Skip to content
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

Add a note to registration page that no account is necessary to access public data #1696

Merged
merged 2 commits into from
Nov 1, 2023

Conversation

yarikoptic
Copy link
Member

I think we start to get an increasing number of registration requests from people who are not intending to upload data to DANDI. This explicit message might help to avoid those registration requests.

@satra
Copy link
Member

satra commented Oct 3, 2023

Yes, this is a trivial thing that should also be released asap.

Copy link
Member

@waxlamp waxlamp left a comment

Choose a reason for hiding this comment

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

This doesn't hurt, but it may not really be enough: by the time they see this message, they've already created their account (albeit, without yet getting it approved, but they've already jumped through the hoop of possibly creating a GitHub account).

Ultimately, the problem here is with an accidental dark pattern we've introduced, where the "log in with github" button doesn't deter people at that stage, and then they are basically on a waterslide that takes them into the kiddie pool of account creation. We should discuss further what we might do about this.

In the meantime, maybe this PR will help.

@waxlamp waxlamp merged commit 40498ec into master Nov 1, 2023
8 of 9 checks passed
@waxlamp waxlamp deleted the enh-no-account-note branch November 1, 2023 17:25
@dandibot
Copy link
Member

dandibot commented Nov 7, 2023

🚀 PR was released in v0.3.60 🚀

@dandibot dandibot added the released This issue/pull request has been released. label Nov 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants