Skip to content

Describe the EHR participant history filter client-side metric #1232

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

Merged
labkey-jeckels merged 2 commits into from
Oct 28, 2026
Merged

Describe the EHR participant history filter client-side metric #1232

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
3d4c766
Describe the EHR participant history filter client-side metric
labkey-jeckels Oct 23, 2026
451de5a
Claude review
labkey-jeckels Oct 24, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions ehr/src/org/labkey/ehr/EHRModule.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,7 @@
import org.labkey.api.security.roles.RoleManager;
import org.labkey.api.settings.AppProps;
import org.labkey.api.settings.OptionalFeatureService;
import org.labkey.api.usageMetrics.SimpleMetricsService;
import org.labkey.api.util.ContextListener;
import org.labkey.api.util.JobRunner;
import org.labkey.api.util.StartupListener;
Expand Down Expand Up @@ -219,6 +220,7 @@ public UpgradeCode getUpgradeCode()
@Override
protected void doStartupAfterSpringConfig(ModuleContext moduleContext)
{
SimpleMetricsService.get().describeClientSideFeatureArea("ehrParticipantHistoryFilter", "Participant history filter use, by filter type: aliveAtCenter or all when a user switches to that filter, idSearch for each completed ID lookup, including ones that match no IDs");

@labkey-bpatel labkey-bpatel Oct 26, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested rewording : "Participant History filter usage by type: Alive at Center, All, or completed ID search, including searches with no matching IDs"

@labkey-jeckels labkey-jeckels Oct 26, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I don't have a strong preference about how these are captured, but aliveAtCenter and idSearch are the terms that will actually show up in metric names, so it seemed useful to capture them in a way that helps guide querying more directly. Let me know if that makes sense to you, or if you still prefer the suggested wording.

EHRService.get().registerReportLink(EHRService.REPORT_LINK_TYPE.project, "View All Projects With Active Assignments", this, DetailsURL.fromString("/query/executeQuery.view?schemaName=ehr&query.queryName=Project&query.activeAssignments/activeAssignments~gt=0"), "Quick Links");
EHRService.get().registerReportLink(EHRService.REPORT_LINK_TYPE.protocol, "View Total Animals Assigned to Each Protocol, By Species", this, DetailsURL.fromString("/query/executeQuery.view?schemaName=ehr&query.queryName=protocolTotalAnimalsBySpecies"), "Quick Links");
EHRService.get().registerReportLink(EHRService.REPORT_LINK_TYPE.assignment, "Find Assignments Overlapping A Date Range", this, DetailsURL.fromString("/query/executeQuery.view?schemaName=study&query.queryName=assignmentOverlapsById"), "Quick Links");
Expand Down
Loading