Skip to content

Akshith: Fix - Most Popular Event - Added Trend indicators , High-Low sorting and clear y-axis description#4595

Open
akshith312 wants to merge 2 commits intodevelopmentfrom
akshith/fix/most-popular-event
Open

Akshith: Fix - Most Popular Event - Added Trend indicators , High-Low sorting and clear y-axis description#4595
akshith312 wants to merge 2 commits intodevelopmentfrom
akshith/fix/most-popular-event

Conversation

@akshith312
Copy link
Copy Markdown
Contributor

Description

Screenshot 2025-12-20 174553 Screenshot 2025-12-20 174614

Related PRS (if any):

This frontend PR is related to the ##3133.

Main changes explained:

  • Added the High - Low and Low - High Sorting Option
  • Added Trend indicators - Increase, Decrease
  • Provided Y-axis description

How to test:

  1. check into current branch
  2. do npm install and npm run start:local to run this PR locally
  3. Clear site data/cache
  4. log as any user
  5. go to dashboard→ /communityportal/reports/event/personalization
  6. Check with Filters and verify if working correctly
  7. Check if Note Description about y-axis is displayed
  8. Toggle to Dark Mode and verify if display is good.

Screenshots or videos of changes:

Screen.Recording.2025-12-20.175646.mp4

Note:

Include the information the reviewers need to know.

@netlify
Copy link
Copy Markdown

netlify Bot commented Dec 20, 2025

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 767a165
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69efb0faff9bd000085ab490
😎 Deploy Preview https://deploy-preview-4595--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@sonarqubecloud
Copy link
Copy Markdown

@akshith312 akshith312 changed the title fix: Added Trend indicators , High-Low sorting and clear y-axis descr… Akshith: Fix - Most Popular Event - Added Trend indicators , High-Low sorting and clear y-axis description Dec 20, 2025
@beblicarl beblicarl self-requested a review February 20, 2026 08:45
@beblicarl
Copy link
Copy Markdown
Contributor

This functionality works as intended, and the code is great

https://www.loom.com/share/18754b60c23642728cd3ce1b0800b4eb

beblicarl
beblicarl previously approved these changes Feb 20, 2026
@akshith312 akshith312 force-pushed the akshith/fix/most-popular-event branch 2 times, most recently from 562bb6e to 5c4094b Compare March 28, 2026 07:13
@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown

@HemanthNidamanuru HemanthNidamanuru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Akshith,

I tested this locally and everything is working fine. The bar chart is rendering correctly in both dark and light mode, trend icons are showing, sorting is working, and the x/y values are displaying as expected.

One thing I could not verify is whether the trend icons match the correct data. Could you please share the mock data you used for the trend indicators so I can cross check them?

Image Image Image Image Image

Copy link
Copy Markdown

@Anusha-Gali Anusha-Gali left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Akshith,

I have reviewed your PR locally and it works as per requirement. However in dark mode, the arrows and - icons do not have the color similar to that of light mode.

Image Image Image Image Image

@one-community one-community added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Apr 6, 2026
@akshith312 akshith312 force-pushed the akshith/fix/most-popular-event branch from 36383d2 to 767a165 Compare April 27, 2026 18:54
@sonarqubecloud
Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants