Skip to content

Vamsi krishna created an interactive list of open issues #1482

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 6 commits into
base: development
Choose a base branch
from
Open

Vamsi krishna created an interactive list of open issues #1482

wants to merge 6 commits into from

Conversation

vamsikrishna1704
Copy link

@vamsikrishna1704 vamsikrishna1704 commented Jun 21, 2025 •
edited
Loading

Description

image
image

Related PRS (if any):

This backend PR is related to the #3683 frontend PR.
…

Main changes explained:

  • Controller handles fetching open issues with or without parameters(projectIds, tag, startDate, endDate)
  • Controller handles updating an issue contents like name or status.
  • Controller handles deleting an issue.
  • Router routes all the requests to respected controller functions.
    …

How to test:

  1. check into current branch
  2. do npm install and npm run build to build and then npm start to run this PR locally
  3. Clear site data/cache on browser
  4. log as admin user
  5. go to Dashboard→ Reports→ Total Construction Summary → Issue Tracking
  6. Verify all the functionalities in both light and dark themes.

Screenshots or videos of changes:

Frontend Demo: (Light theme)

Dashboard.-.Vamsi.Krishna.Owner.-.Profile.1.-.Microsoft.Edge.2025-06-21.15-33-17.mp4

Frontend Demo:(Dark theme)
https://github.com/user-attachments/assets/6796d55c-c58f-4a1a-b427-327bc6f41325

@vamsikrishna1704 vamsikrishna1704 mentioned this pull request Jun 21, 2025
Open
@vamsikrishna1704 vamsikrishna1704 added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Jun 21, 2025
@Venk-rgb Venk-rgb self-assigned this Jun 22, 2025
Venk-rgb
Venk-rgb approved these changes Jun 22, 2025
View reviewed changes
Copy link

@Venk-rgb Venk-rgb left a comment

Choose a reason for hiding this comment

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

Tested with Frontend PR #3683. Everything working as expected. Good job.

PR.3683.-.FE.mov

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

@Venk-rgb Venk-rgb Venk-rgb approved these changes

Assignees

@Venk-rgb Venk-rgb

Labels
High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vamsikrishna1704 @Venk-rgb