Skip to content

Comments

pathogens/files: Show restricted warning in modal#1313

Merged
victorlin merged 5 commits intomasterfrom
victorlin/restricted-warning
Feb 9, 2026
Merged

pathogens/files: Show restricted warning in modal#1313
victorlin merged 5 commits intomasterfrom
victorlin/restricted-warning

Conversation

@victorlin
Copy link
Member

@victorlin victorlin commented Feb 6, 2026

follow-up to #1311 (comment)

This PR contains 4 prep commits + 1 main commit. Message from main commit:

This mirrors the existing warning on the IndividualResource component.

The Terms of Use link on the original page is blocked by the modal, so I've copied the paragraph with the link into the modal. I chose not to copy all the other data use language to keep the overall modal description text to a minimum.

image

Checklist

To be used by the following commit.
This provides a cleaner separation between logical detection and
context-specific text.
Preparing to reuse this logic in another context.
This simplifies some logic and makes it easier to add more metadata in
the future.
This mirrors the existing warning on the IndividualResource component.

The Terms of Use link on the original page is blocked by the modal, so
I've copied the paragraph with the link into the modal. I chose not to
copy all the other data use language to keep the overall modal
description text to a minimum.
@victorlin victorlin self-assigned this Feb 6, 2026
@nextstrain-bot nextstrain-bot temporarily deployed to nextstrain-s-victorlin--oaho5s February 6, 2026 22:48 Inactive
@victorlin victorlin merged commit 96ac152 into master Feb 9, 2026
10 checks passed
@victorlin victorlin deleted the victorlin/restricted-warning branch February 9, 2026 17:50
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