Dennis Kigen
Dennis Kigen
Also, circling back to the configuration issue, do we still want to make this the default behaviour out of the box for the refapp with the GP as the only...
@ibacher @mogoodrich I think this is ready to be reviewed anew
I've set this to draft because @dkayiwa is exploring a potential regression in the backend that's causing discontinued orders to have the incorrect `action` property (`NEW` instead of `DISCONTINUE`).
State of things following the latest commits (forgive the slow load times - they're owing to really bad internet): https://github.com/openmrs/openmrs-esm-patient-chart/assets/8509731/46f0d1cf-1335-4172-82bf-9a338d45bc1a
@mogoodrich I think Ian means we should add this as a configurable property to the config schema for the chart app.
Yeah, makes sense. Thanks!
Is the bundle size change [estimate](https://github.com/openmrs/openmrs-esm-core/pull/969#issuecomment-2045323229) accurate, @ibacher, in the sense that this change reduces the bundle size rather than inflating it?
~~Should it be straightforward to test locally? Getting these placeholders instead of the actual icons:~~  Well, ignore all of that. I hadn't read the...
Sorry I'm getting to this late @trevor-james-nangosha @ibacher, but this is a mockup that Ciaran's thrown together for what this error notification could look like: ![Screenshot 2024-02-15 at 11 50...
Would clicking the `Reload` button just invoke `window.location.reload()` or would it do something else?