-
Notifications
You must be signed in to change notification settings - Fork 0
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.
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
Require login for downloading data #283
Open
1 task
Comments
|
MillenniumFalconMechanic
added a commit
that referenced
this issue
Feb 22, 2025
frano-m
pushed a commit
that referenced
this issue
Feb 24, 2025
MillenniumFalconMechanic
pushed a commit
that referenced
this issue
Feb 24, 2025
MillenniumFalconMechanic
added a commit
that referenced
this issue
Feb 26, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Need
A configurable "login required for downloading data" option is needed for prompting users to login before initiating a download. Specifically, a login-required popup is displayed on click of download if user is not authenticated and the "login required" flag is set for an app.
Login is not required to view pages that display download buttons (e.g. on the "Choose Export Method" page, login is only required on click of a download option not to view the actual page).
User should remain in the same flow as before login. For example:
The download actions that require authentication are:
Request curl Command (from cohort Choose Export Method)Request File Manifest (from cohort Choose Export Method)Analyze in Terra (from cohort Choose Export Method)Metadata download and copy (from entity Metadata tab)Note: this list may require updating once Make the general and dataset-details Export panes consistent is completed.
No change is required to the regular login use case unless updates to existing implementation are required to facilitate reuse with this ticket's use case.
See design ticket for rationale and mock for implementation.
Approach
The text was updated successfully, but these errors were encountered: