-
Notifications
You must be signed in to change notification settings - Fork 269
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.
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
Add the In Progress Filter to Pathology Dashboard and Cytology Dashboard #1369
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is not advisable to remove the value field unless there is a plan to handle display names for statuses differently, as it is critical for presenting user-friendly names in the UI.
And also try to run the cd frontend then npm run format to format the changes |
Hey thanks for mentioning but I ran it already |
Will do the changes |
Hi @mozzy11, I’d love to hear your feedback on this when you get a chance. Let me know if there’s anything I can improve or adjust! |
Thanks @vsvishalsharma . |
@mozzy11 I have removed the hardcode statuses |
Great job @vsvishalsharma |
Pull Requests Requirements
Issue number if applicable.
documentation.
Summary
Changed the setStatusfilter function to setup the In Progress filter with the following
Screenshots
#1370 OGC 16
To DO: