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.

Sign up for GitHub

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

Jump to bottom

[LabKey R&D - EHR] Migrate EHR inline event handlers #5330

Open
10 tasks done
labkey-adam opened this issue Mar 14, 2024 · 0 comments
Open
10 tasks done

[LabKey R&D - EHR] Migrate EHR inline event handlers #5330

labkey-adam opened this issue Mar 14, 2024 · 0 comments
Assignees
@ankurjuneja

Comments

@labkey-adam
Copy link
Contributor

labkey-adam commented Mar 14, 2024 •
edited by ankurjuneja
Loading

Known Inline Handlers

Tasks

  • Eliminate inline handlers from SnapshotPanel.js. ehrModules & nircEHRModules @ankurjuneja
  • Eliminate inline handlers from SnapshotPanel.js and snprcOverrides.js. onprcEHRModules and snprcEHRModules @ankurjuneja
  • Migrate inline handlers out of EHRCustomizers and various display columns in ehrModules. See spreadsheet for full list @ankurjuneja
  • Migrate inline handlers out of EHRCustomizers and various display columns in onprcModules. See spreadsheet for full list @ankurjuneja
  • Decide what to do about WebUtils/lk-table.jsp: migrate? ignore? eliminate? @ankurjuneja
  • JHU ClinPathRuns.js @ankurjuneja
  • MenuBarHoverNavigation.js used by EHR, JHU, SNPRC quickSearch. Should we reconcile with ONPRC quickSearch approach? Use UIComponents navigation.js instead? @ankurjuneja
  • LabDevKitModules/Printer.js has a couple inline onclick handlers. These could be migrated, but I see no usages of this JS file. Please confirm and delete. @ankurjuneja
  • Code review
  • TC Review and merge
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ankurjuneja ankurjuneja

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@labkey-adam @labkey-martyp @ankurjuneja