Skip to content

Conversation

@labkey-martyp
Copy link
Contributor

@labkey-martyp labkey-martyp commented Jul 20, 2025

Rationale

This sets up the core EHR link crawler tests and fixes a number of broken links and CSP violations.

Related Pull Requests

Changes

  • Add cases dataset to avoid errors in core EHR components on test server
  • Remove invalid lookup (error_types) from list of editable lookups
  • Remove invalid links from EHR admin page
  • Remove invalid housing report link from more reports
  • Register bootstrap 3.3.6 style and fonts as allowed CSP sources
  • Fix various NPEs found using sporadic test data
  • Add nonces to necropsy schedule
  • Update WNPRC test to do more WNPRC set up to mimic the production server

Comment on lines +404 to +405
links.add("wnprc_ehr-PerDiems.view");
links.add("wnprc_ehr-WaterCalendar.view");
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please explain why these actions are skipped

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

Comment on lines 413 to 422
links.add("query-executeQuery.view?query.queryName=Current Colony Condition&schemaName=study");
links.add("query-executeQuery.view?schemaName=col_dump&query.queryName=mysql_check");
// links.add("ehr-dataEntryFormDetails.view?formtype=Enter%20Water%20Orders&taskid=");
links.add("wnprc_ehr-UnscheduleBCReports.view");
links.add("wnprc_ehr-ScheduleBCReports.view");
links.add("wnprc_ehr-NecropsySchedule.view");
links.add("Documentation/Admin/project-begin.view");
links.add("Documentation/Data Management/project-begin.view");
links.add("Development Notes/project-begin.view");
links.add("/Logs/project-begin.view");
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please explain why these actions are skipped

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@labkey-martyp labkey-martyp merged commit b405a2f into release25.7-SNAPSHOT Aug 3, 2025
6 of 8 checks passed
@labkey-martyp labkey-martyp deleted the 25.7_fb_ehr_link_crawler branch August 3, 2025 23:23
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.

3 participants