(feat) O3-2801 - Add icons to patient chart left nav #2121
Conversation
|
@denniskigen @gracepotma @CynthiaKamau I have made the changes. I had a problem with the last pull request with the build but this one should work just fine. KIndly review |
|
Asante SANA @Munyua123!! LGTM <3 |
packages/esm-patient-common-lib/src/dashboards/DashboardExtension.tsx
Outdated
Show resolved
Hide resolved
packages/esm-patient-common-lib/src/dashboards/DashboardExtension.tsx
Outdated
Show resolved
Hide resolved
packages/esm-patient-common-lib/src/dashboards/dashboard-extension.scss
Outdated
Show resolved
Hide resolved
packages/esm-patient-common-lib/src/dashboards/dashboard-extension.scss
Outdated
Show resolved
Hide resolved
|
@ibacher Happy New Year. I'm sorry for the delay in solving the conflicts you raised, I have been able to solve some of the conflicts you raised above. Could you review and revert with any changes? |
…nt-chart into patient-chart-icons
…nt-chart into patient-chart-icons
…tension then passing the link using the dashboard meta
…enmrs-esm-patient-chart into patient-chart-icons
|
@ibacher I have added the Icon as a prop to the dashboard link configuration and dashboard extension, where I have put up the Maybeicon as you had instructed and passed the icons to all the menu titles from the dashboard meta. The icons for the appointments and billing history since they are in their modules / esm once this is merged I can be able to add their icons from the modules. Could you review the above? Thank you |
Requirements
Summary
Screenshots
Related Issue
https://openmrs.atlassian.net/browse/O3-2801Other