Skip to content

Conversation

@labkey-martyp
Copy link
Contributor

Rationale

There are still a couple places in EHR, such as site search results, that show the base study participant view. This PR overrides that view to use EHR participant view. Goes to base EHR participantView but this action should be overridden to custom participantViews as necessary.

Changes

  • participant HTML override with redirect

@bbimber
Copy link
Collaborator

bbimber commented Nov 26, 2025

@labkey-martyp: I'm not sure of the background, but is there any reason not to do this as a java action in EhrController? That would be a cleaner redirect without needing to load anything on the client, just to immediately redirect and load something else.

If there is a need for center-specific modules to be able to override it, could that java code check whether any active module has a participant.html file and defer to that if present?

@labkey-martyp
Copy link
Contributor Author

@bbimber Unfortunately, the study module is looking specifically for a custom module html view to override the study's participant view. An action in EHR controller will not override the base study participant view. There's a ticket open and some discussion to allow an action override in study configuration of the participant view. This is a little clunky but gets the basic functionality for a fairly rare use case.

@bbimber
Copy link
Collaborator

bbimber commented Dec 1, 2025

OK

@labkey-martyp labkey-martyp merged commit 1b6e551 into develop Dec 1, 2025
7 of 9 checks passed
@labkey-martyp labkey-martyp deleted the fb_participant_redirect branch December 1, 2025 14:22
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.

4 participants