-
Notifications
You must be signed in to change notification settings - Fork 3
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
23.11 fb surgical rounds observation #1108
base: release23.11-SNAPSHOT
Are you sure you want to change the base?
23.11 fb surgical rounds observation #1108
Conversation
…on input columns.
WARNING: This PR appears to have the default title generated by GitHub. Please use something more descriptive. |
…vation input columns.
…vation input columns.
…vation input columns.
…n new reportin columns.
…n new reportin columns.
…n new reportin columns.
…n new reportin columns.
…n new reportin columns.
…rvation # Conflicts: # onprc_ehr/resources/schemas/dbscripts/sqlserver/onprc_ehr-23.015-23.016.sql
@Ohsudev I believe this commit fixes the issue with the multi-select. Please check it out when you get a chance. |
…ghtlighting of multislect combo box selections.
…on' into 23.11_fb_SurgicalRoundsObservation # Conflicts: # onprc_ehr/resources/web/onprc_ehr/grid/ObservationsRowEditorGridPanel.js
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.
Let me know when you're ready to merge this in and I'll merge in the associated EHR PR.
Please add the new dataset columns to datasets_metadata.xml
Most of the code that I have added, and the code you modified should all have been added to
the ONPRC_EHR side of things.
You shouldn’t have to merge any code on the EHR side of things.
If I failed to convert a file to the ONPRC_EHR version, please let me know as soon as possible.
Always appreciated your help past, and present.
In addition, I have added the new set of columns to the file “datasets_metadata.xml” as you had instructed.
Thanks,
Raymond
From: Marty Pradere ***@***.***>
Sent: Wednesday, December 18, 2024 11:21 AM
To: LabKey/onprcEHRModules ***@***.***>
Cc: Raymond Blasa ***@***.***>; Mention ***@***.***>
Subject: [EXTERNAL] Re: [LabKey/onprcEHRModules] 23.11 fb surgical rounds observation (PR #1108)
@labkey-martyp commented on this pull request.
Let me know when you're ready to merge this in and I'll merge in the associated EHR PR.
Please add the new dataset columns to datasets_metadata.xml
—
Reply to this email directly, view it on GitHub<https://urldefense.com/v3/__https:/github.com/LabKey/onprcEHRModules/pull/1108*pullrequestreview-2512677829__;Iw!!Mi0JBg!NePrGXzu_F2v68HKpQJLT_xAvxWHWP_gV9pIc6YFGrMYEg9oujEKyE8oYuYc6SikWVACYCB2MyqhByvZHKZzGIw$>, or unsubscribe<https://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/ASHU2YDZH7ZGK2HZHZNSEBL2GHDH7AVCNFSM6AAAAABMUYIGFGVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDKMJSGY3TOOBSHE__;!!Mi0JBg!NePrGXzu_F2v68HKpQJLT_xAvxWHWP_gV9pIc6YFGrMYEg9oujEKyE8oYuYc6SikWVACYCB2MyqhByvZz5r0Mq0$>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
@Ohsudev Got it, I see you copied it over. Looks good. |
…differentiate between surgery and clincal entries.
…differentiate between surgery and clincal entries.
…differentiate between surgery and clincal entries.
…differentiate between surgery and clincal entries.
Modified for Surgical Rounds input form to include new Clinical Observation input columns.