-
Notifications
You must be signed in to change notification settings - Fork 347
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
Generalize Import and Export + Add create asset button on asset list page + Fix patients page padding #4310
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
✅ Deploy Preview for care-egov-staging ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
👋 Hi, @rithviknishad, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
@rithviknishad PR LGTM, marking it for Team discussion required for final UI Approval. @rithviknishad check the cypress deplyment |
@nihal467 I've updated the cypress test for external results |
@nihal467 this pr was already discussed in yesterday's EOD I guess, wasn't it already approved @rithviknishad? |
@khavinshankar yes, kindly review it :) |
Kudos, SonarCloud Quality Gate passed!
|
Proposed Changes
Generalization of Import / Export
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Create assets directly from the Asset list page (@Ashesh3)
Fixes inconsistent padding present in patients page
So many variations of Import and Export buttons now became just one variation
Merge Checklist