-
Notifications
You must be signed in to change notification settings - Fork 187
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
Fix 705 rewriting patient report screen #714
Fix 705 rewriting patient report screen #714
Conversation
hey @mozzy11 If any changes required let me know 🙂 Thank You |
Thanks @adityadeshlahre . generally Looks ggod. |
cc8637e
to
ea004aa
Compare
Hey @mozzy11 👋 this is one way in which we could implement the routing to render different if any changes required let me know 🙂 Thank You Video |
Hello @adityadeshlahre , generally Looks good. |
@adityadeshlahre can you adress the above comments and also fix the merge conflicts ?? |
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.
seeabove comments
👋 Hi, @adityadeshlahre, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
965cb6f
to
29d07a4
Compare
Hey @mozzy11 i have updated this PR fixed all conflicts! |
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.
you aded a path ActivityReport
with no Route handling it . can you remove that ??
👋 Hi, @adityadeshlahre, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
Hey @mozzy11 you were the only one who asked for it 🙂 |
851c29a
to
fedfa1f
Compare
Hey @mozzy11 i have made the requested changes 🙂 |
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.
Thanks @adityadeshlahre
fixes #705
Rewritten the Patient Status Report screen.
Added Translation and Created New Component .
Video
Images FR lang