Skip to content

Register Process Flow Step URIs in stage mapping#380

Merged
ozancinar merged 1 commit intomainfrom
feature/workflow-step-mapping-v2
Apr 20, 2026
Merged

Register Process Flow Step URIs in stage mapping#380
ozancinar merged 1 commit intomainfrom
feature/workflow-step-mapping-v2

Conversation

@marvinm2
Copy link
Copy Markdown
Contributor

Summary

  • Extends stage_mapping in app.py with the 5 new VHP glossary URIs (VHP0000153158) for the Safety Assessment Workflow Steps: Chemical Characteristics and Hazard Identification, Exposure, Toxicokinetics, Toxicodynamics, Adverse Outcome.
  • Extends STAGE_EXPLANATIONS with the full tooltip definitions for each new label (text mirrors the button title attributes already present in templates/tools/tools.html:43-56).
  • Legacy entries are kept alongside the new ones so the data and methods pages keep working until their upstream data sources migrate.

Pairs with VHP4Safety/cloud#290 (merged) which retagged each tool's stage field with the new URIs. Once service_index.json regenerates and this PR lands, the /tools filter dropdown will show exactly the 5 canonical labels plus "Other", matching the 5 workflow-step buttons shown above it.

Supersedes #379.

Test plan

  • After CI regenerates cap/service_index.json upstream, load /tools locally and confirm the "Safety Assessment Workflow Step" dropdown lists the 5 new labels + Other (no raw URL strings).
  • Hover any dropdown item to confirm the tooltip renders.
  • Confirm /data and /methods pages still render stage filters without regression.

Extend stage_mapping with the 5 new VHP glossary URIs (VHP0000153-158)
so the /tools filter dropdown displays the Safety Assessment Workflow
Step labels instead of raw URL strings once the upstream cloud repo
data migrates to the new URIs.

Also extend STAGE_EXPLANATIONS with the full tooltip definitions for
each new label. Legacy entries are kept for the data and methods pages
until their data sources migrate.
@ozancinar ozancinar merged commit b6093c2 into main Apr 20, 2026
2 checks passed
@ozancinar
Copy link
Copy Markdown
Contributor

The pull request has been merged and the updates have been applied to the live version. Thanks.

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.

2 participants