Skip to content

Alisha - feature : implementation of BMDashboard grouped bar-graph showing Issues created vs Resolved of projects frontend#4272

Open
alishawalunj wants to merge 7 commits intodevelopmentfrom
alisha/feature/issues-bar-graph
Open

Alisha - feature : implementation of BMDashboard grouped bar-graph showing Issues created vs Resolved of projects frontend#4272
alishawalunj wants to merge 7 commits intodevelopmentfrom
alisha/feature/issues-bar-graph

Conversation

@alishawalunj
Copy link
Copy Markdown
Contributor

Description

353E032F-B528-448A-A36C-850AE2284C74
CCF459ED-89FA-49A3-8ADF-CDC3919A3A5E
DFDAD638-D876-4A61-A841-968AE460BCB7

Related PRS (if any):

This frontend PR is related to the #1853 backend PR.

Main changes explained

  • Designed and implemented the IssueGraph page to display filters, tiles, and the chart.
  • Configured Redux reducer and actions for the IssueGraph.

How to test

  1. Check out the current branch.
  2. Run npm install and start the application locally.
  3. Clear site data/cache in your browser.
  4. Log in as an admin user.
  5. Navigate to: http://localhost:5173/bmdashboard/issueGraph.
  6. Verify that by default, the graph and tiles display data for the last 8 weeks.
  7. Change the filter values and verify that the tiles and graph update accordingly.
  8. Start Date: Ensure it can be selected only within a 12-week range from today; other dates should be disabled.
  9. End Date: Ensure it cannot be later than today; future dates should be disabled.
  10. Verify the style and display of the page match the requirements mentioned in the description
  11. Verify that this feature works correctly in dark mode.

Screenshots or videos of changes:

REC-20251024212017.mp4

@netlify
Copy link
Copy Markdown

netlify Bot commented Oct 25, 2025

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 417a14d
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69f590ef30e2fe0008c4e18e
😎 Deploy Preview https://deploy-preview-4272--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

@alishawalunj alishawalunj added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Oct 25, 2025
@sonarqubecloud
Copy link
Copy Markdown

akshith312
akshith312 previously approved these changes Oct 25, 2025
Copy link
Copy Markdown
Contributor

@akshith312 akshith312 left a comment

Choose a reason for hiding this comment

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

Tested PR locally.
Logged in as Admin user.
Navigated to bmdashboard. Chart renders as expected, filters work as expected and doesn't allow to choose a date more than 12 weeks from current day.
Works in dark mode as well.
image
No Option to select prior to 12 weeks from today.
image
Date Filtering:
image
image

@apoorvajainrp21
Copy link
Copy Markdown
Contributor

Tested the PR as explained. Iencountered the error as shown below, please suggest any changes in testing if required:
image

Aswin20010
Aswin20010 previously approved these changes Oct 27, 2025
Copy link
Copy Markdown

@Aswin20010 Aswin20010 left a comment

Choose a reason for hiding this comment

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

IssueGraph page functions as expected — filters, tiles, and chart update dynamically. Date range validation works correctly, and the layout aligns with design requirements in both light and dark modes. Implementation is stable and meets feature specifications.

Screenshot 2025-10-27 at 2 41 31 PM Screenshot 2025-10-27 at 2 42 03 PM Screenshot 2025-10-27 at 2 42 17 PM Screenshot 2025-10-27 at 2 42 57 PM

@beblicarl
Copy link
Copy Markdown
Contributor

This functionality works as intended and the code is great

image image image image

@beblicarl beblicarl self-requested a review November 21, 2025 11:10
beblicarl
beblicarl previously approved these changes Nov 21, 2025
@alishawalunj alishawalunj force-pushed the alisha/feature/issues-bar-graph branch from e83b92f to e3f8d98 Compare December 27, 2025 05:14
@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

@sayali-2308 sayali-2308 left a comment

Choose a reason for hiding this comment

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

Image
  • Checked out branch: alisha/feature/issues-bar-graph
  • Backend branch: alisha/feature/issue-analytics-grouped-bar-graph
  • Tested on: Windows, Chrome browser
  • Both servers running locally (Backend: 4500, Frontend: 5173)

⚠️ Issue Found:
Bug: End Date doesn't auto-populate when selecting weeks from dropdown
Steps to reproduce:

  1. Select "Last 8 Weeks" or "Last 12 Weeks" from Weeks dropdown
  2. Click "Go" button
  3. Expected: End Date auto-populates to today's date (January 30, 2026)
  4. Actual: End Date remains empty (dd-mm-yyyy placeholder)
  5. Workaround: Manually selecting a Start Date causes End Date to update
    Note: Start Date correctly auto-populates based on weeks selected. Only End Date has this issue.

Functionality Tested:

  1. Navigated to /bmdashboard/issueGraph - page loads correctly
  2. Default display shows "Last 8 Weeks" filter
  3. All 4 KPI tiles display correctly (Total Issues, New Issues This Week, Resolved Issues, Avg. Resolution Time)
  4. Chart title "Issues Created vs. Resolved" displays correctly
  5. Start Date validation works - cannot select dates more than 12 weeks ago (tested October 2025, correctly disabled)
  6. End Date validation works - cannot select future dates (tested dates after January 30, 2026, correctly disabled)
  7. Weeks dropdown works with options: Last 4 Weeks, Last 8 Weeks, Last 12 Weeks
  8. Filter changes update the page when "Go" button is clicked
  9. Dark mode styling works correctly - proper contrast, readable text, tiles styled appropriately
  10. Backend APIs tested in Postman:
    • GET /api/issues/summary - returns correct JSON structure
    • GET /api/issues/trends - returns correct JSON structure with metadata
  11. No console errors observed (normal browser warnings only)

The End Date auto-population logic should be reviewed to ensure it updates when the "Weeks" dropdown is changed and "Go" is clicked, similar to how Start Date updates.

@alishawalunj
Copy link
Copy Markdown
Contributor Author

alishawalunj commented Feb 1, 2026

4. dd-mm-yyyy

Image * Checked out branch: `alisha/feature/issues-bar-graph` * Backend branch: `alisha/feature/issue-analytics-grouped-bar-graph` * Tested on: Windows, Chrome browser * Both servers running locally (Backend: 4500, Frontend: 5173)

⚠️ Issue Found: Bug: End Date doesn't auto-populate when selecting weeks from dropdown Steps to reproduce:

  1. Select "Last 8 Weeks" or "Last 12 Weeks" from Weeks dropdown
  2. Click "Go" button
  3. Expected: End Date auto-populates to today's date (January 30, 2026)
  4. Actual: End Date remains empty (dd-mm-yyyy placeholder)
  5. Workaround: Manually selecting a Start Date causes End Date to update
    Note: Start Date correctly auto-populates based on weeks selected. Only End Date has this issue.

Functionality Tested:

  1. Navigated to /bmdashboard/issueGraph - page loads correctly

  2. Default display shows "Last 8 Weeks" filter

  3. All 4 KPI tiles display correctly (Total Issues, New Issues This Week, Resolved Issues, Avg. Resolution Time)

  4. Chart title "Issues Created vs. Resolved" displays correctly

  5. Start Date validation works - cannot select dates more than 12 weeks ago (tested October 2025, correctly disabled)

  6. End Date validation works - cannot select future dates (tested dates after January 30, 2026, correctly disabled)

  7. Weeks dropdown works with options: Last 4 Weeks, Last 8 Weeks, Last 12 Weeks

  8. Filter changes update the page when "Go" button is clicked

  9. Dark mode styling works correctly - proper contrast, readable text, tiles styled appropriately

  10. Backend APIs tested in Postman:

    • GET /api/issues/summary - returns correct JSON structure
    • GET /api/issues/trends - returns correct JSON structure with metadata
  11. No console errors observed (normal browser warnings only)

The End Date auto-population logic should be reviewed to ensure it updates when the "Weeks" dropdown is changed and "Go" is clicked, similar to how Start Date updates.

@sayali-2308
Please read the steps in the description and the requirements carefully.
Why is the “Go” button being clicked after selecting an option from the Weeks dropdown?
The “Go” button is meant to be used only when custom start and end dates are selected.
The Weeks dropdown works on its own and triggers the backend query immediately when an option is selected.
These are two separate and intentional ways to fetch data, not a bug.
The date placeholder behavior depends on the browser:

  • In Chrome, it shows dd/mm/yyyy
  • In Safari, today’s date is shown by default

This is standard browser behavior and cannot be controlled by the application.
The dropdown was added for quick timeline selection.
This is a misunderstanding of the requirements and incorrect use of the UI, not an issue with the implementation.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Feb 1, 2026

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 Alisha,

I have reviewed your PR locally and the chart does not load. I am assuming it is because there is no data for the past 4,8,12 weeks. However in backed when selecting dates such as the one's you mentioned, i do get the KPI values in postman but unfortunately i am unable to select those values in frontend since those dates are blocked in the calendar.

Image Image

@alishawalunj alishawalunj force-pushed the alisha/feature/issues-bar-graph branch from 21f8488 to 443e795 Compare May 2, 2026 05:18
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 2, 2026

@alishawalunj
Copy link
Copy Markdown
Contributor Author

Resolved the data issue by updating the dates and fixed labelList dark mode issue.

REC-20260502015554.mp4

@alishawalunj alishawalunj requested a review from sayali-2308 May 2, 2026 05:59
@alishawalunj alishawalunj requested a review from Anusha-Gali May 2, 2026 05:59
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.

7 participants