Skip to content

[codex] Show restricted download exclusion count#6400

Draft
theosanderson wants to merge 1 commit into
mainfrom
dlcount
Draft

[codex] Show restricted download exclusion count#6400
theosanderson wants to merge 1 commit into
mainfrom
dlcount

Conversation

@theosanderson
Copy link
Copy Markdown
Member

@theosanderson theosanderson commented May 11, 2026

Summary

Show a notice in the download modal when restricted-use data is excluded. If the user leaves "No, only download open data" selected, the modal now states how many restricted sequences will not be included in the download.

Implementation

  • Adds an aggregated LAPIS query grouped by dataUseTerms for the active download filter.
  • Passes the restricted sequence count into the download dialog.
  • Hides the notice when restricted data is included.

Validation

  • npm test -- --run src/components/SearchPage/DownloadDialog/DownloadDialog.spec.tsx src/components/SearchPage/SearchFullUI.spec.tsx
  • npx eslint src/components/SearchPage/DownloadDialog/DownloadDialog.tsx src/components/SearchPage/SearchFullUI.tsx src/components/SearchPage/DownloadDialog/DownloadDialog.spec.tsx
  • npx prettier --check src/components/SearchPage/DownloadDialog/DownloadDialog.tsx src/components/SearchPage/SearchFullUI.tsx src/components/SearchPage/DownloadDialog/DownloadDialog.spec.tsx
  • npm run check-types

🚀 Preview: Add preview label to enable

@theosanderson-agent theosanderson-agent added the preview Triggers a deployment to argocd label May 11, 2026
@claude claude Bot added the website Tasks related to the web application label May 11, 2026
@theosanderson theosanderson added preview Triggers a deployment to argocd and removed preview Triggers a deployment to argocd labels May 11, 2026
@theosanderson theosanderson added the preview Triggers a deployment to argocd label May 21, 2026
@anna-parker anna-parker removed the preview Triggers a deployment to argocd label May 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

website Tasks related to the web application

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants