-
Notifications
You must be signed in to change notification settings - Fork 486
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
"Log Update" Consistency #8283
"Log Update" Consistency #8283
Conversation
@renoseHarsh is attempting to deploy a commit to the Open Healthcare Network Team on Vercel. A member of the Team first needs to authorize it. |
✅ Deploy Preview for care-egov-staging ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
src/Components/Facility/ConsultationDetails/Events/EventsList.tsx
Outdated
Show resolved
Hide resolved
I removed changes that could be handled by backend |
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.
There are other routes in this file that are still having the term 'daily rounds' in the paths. Let's change that too.
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.
Should I also change the 'daily_rounds' or is that different from 'daily-rounds'
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.
@renoseHarsh Yes refactor all instances of daily_rounds, daily-rounds, Daily Rounds
👋 Hi, @renoseHarsh, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
👋 Hi, @renoseHarsh, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
@renoseHarsh Resolved the conflicts and merged develop, Make sure that you pull, before you start working on it. Update the PR once it's ready to be reviewed. |
Closing this PR due to lack of recent progress. If you are re-opening the PR, please make sure the strings are moved to I18n |
Proposed Changes
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Merge Checklist