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

Auth: add a cookie to pin user to beta instance #11249

Open
agjohnson opened this issue Mar 28, 2024 · 1 comment
Open

Auth: add a cookie to pin user to beta instance #11249

agjohnson opened this issue Mar 28, 2024 · 1 comment
Labels
Accepted Accepted issue on our roadmap Feature New feature Improvement Minor improvement to code

Comments

@agjohnson
Copy link
Contributor

On login or maybe on use of the beta dashboard, we should set a cookie or session variable to redirect the user to the beta dashboard whenever they log in. The notification links will be temporary

@agjohnson agjohnson added Improvement Minor improvement to code Feature New feature Accepted Accepted issue on our roadmap labels Mar 28, 2024
@humitos
Copy link
Member

humitos commented Apr 25, 2024

On login or maybe on use of the beta dashboard

I think this won't work since we need the beta=true cookie to be on readthedocs.org not on the beta dashboard domain. However, I suppose that we can set the cookie when the user clicks on the notification to go to the new dashboard from the old dashboard

Screenshot_2024-04-25_12-00-33

and remove when they click on the notification from the new dashboard.

Screenshot_2024-04-25_12-01-44

I suppose we will need to modify that link with something like readthedocs.org/?cookie=remove and remove the cookie from Django since it's also a different domain.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted Accepted issue on our roadmap Feature New feature Improvement Minor improvement to code
Projects
Status: Planned
Development

No branches or pull requests

2 participants