-
Notifications
You must be signed in to change notification settings - Fork 206
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) Pass patient object to the banner action buttons #1158
Conversation
Size Change: -85 kB (-1.46%) Total Size: 5.73 MB
ℹ️ View Unchanged
|
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.
LGTM. Thanks, @vasharma05.
@vasharma05 could you please file the corresponding Patient Chart PR when you can? |
@vasharma05 |
Requirements
feat
,fix
, orchore
, among others). See existing PR titles for inspiration.For changes to apps
If applicable
Summary
This is a follow-up on openmrs/openmrs-esm-patient-chart#2015, where the patient banner action buttons were updated to accept the
patient
object, but weren't passed from the ExtensionSlot at the Patient Banner Action Menu. This takes thepatient
object from the patient banner and passes it down to the Action buttons.A follow-up PR in https://github.com/openmrs/openmrs-esm-patient-chart will be created to pass down the
patient
object from the patient banner to the Action Menu.Screenshots
None
Related Issue
None
Other