Ian
Ian
The build here is failing not because of anything specific to do with versions, but because the OCL package that this tries to add is broken, in the sense that...
Code seems fine. However, I think we want to make this more prominent. Instead of just saying "— DISCONTINUED" at the end of the block, I think we should put...
It shouldn't be possible to remove the OpenMRS logo from the Powered By slot at all.
So two other things need to happen for this PR to work: 1. We need to actually register this extension with the extension system (try renaming the extension to global-nav-group...
> By registering, do you mean creating a store and registering it in the store as they did in the common-lib? No. I mean actually registering the extension with the...
> Can you confirm that we want to add support for printing lab orders here? I think it's a reasonable place for printing lab results, but it probably isn't the...
> The list should default to showing 20 visits instead of 10 (must-have but not a blocker for merging this PR) I think we should be _very_ careful about specifying...
I think we explicitly moved away from the model where everything was in the patient summary, so for now I wouldn't. We can do that later if we decide that's...
No, it does not. I think this does something else. @suubi-joshua How can I test this? What does it do? Why is there anything about config schemas in this? The...