Skip to content
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

Changes to make this run in the RefApp #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ibacher
Copy link

@ibacher ibacher commented Aug 29, 2019

I'm not sure if this is of interest or not, but these are changes I've made to get this module running in the RefApp using CIEL concepts where possible.

The change concepts are:

Concept Old UUID New UUID
Convulsive Seizure 3cce938e-26fe-102b-80cb-0017a47871b2 206AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
Respiratory Rate 3ceb11f8-26fe-102b-80cb-0017a47871b2 5242AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
Blood Oxygen Saturation 3ce9401c-26fe-102b-80cb-0017a47871b2 5092AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
Heart Rate (Pulse) 3ce93824-26fe-102b-80cb-0017a47871b2 5087AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
Systolic Blood Pressure 3ce934fa-26fe-102b-80cb-0017a47871b2 5085AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
Diastolic Blood Pressure 3ce93694-26fe-102b-80cb-0017a47871b2 5086AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
Temperature (ºC) 3ce939d2-26fe-102b-80cb-0017a47871b2 5088AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
Weight (kg) 3ce93b62-26fe-102b-80cb-0017a47871b2 5089AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
Impaired Breathing 3cedf31e-26fe-102b-80cb-0017a47871b2 142373AAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
Shock 911c064e-5247-4017-a9fd-b30105c36052 112989AAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
Dyspnea 3cf1a95a-26fe-102b-80cb-0017a47871b2 141600AAAAAAAAAAAAAAAAAAAAAAAAAAAAAA

The bulk of the important changes are in EDTriageAppActivator and EDTriageEditPatientPageController.

Apart from the changing concept IDs, I've tried to keep things as compatible as possible with the current version of the module, although I did add some logic to the EDTriageEditPatientController to make page flow a bit more sensible.

@mogoodrich
Copy link
Member

Thanks @ibacher ... I didn't realize this was going on, but would be great to get this working with the generic reference app... were you planning on using this in particular?

One challenge is that we are actively using the app with the the existing concepts, so it's hard to change the uuids... one thing I would suggest would be to change the whole thing to support using concept mappings instead of uuids... we could even great a concept source just for this module, and implementations apply the appropriate mappings to the concepts they'd want to use.

@ibacher
Copy link
Author

ibacher commented Aug 30, 2019

@mogoodrich I'm not entirely sure if we're planning on using this or not. The project I'm working on is currently a bit devoid of feedback. That said, I was told this was something that may be of interest to them.

And, yeah, I didn't like the whole changing UUIDs thing but I also didn't like that they're hard-coded in the Javascript... I'll have a look at making this work with concept mappings instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants