-
Notifications
You must be signed in to change notification settings - Fork 6
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
Point community to new dashboard #332
Conversation
We are already migrated on community, so I think we can start pointing people to the new dashboard already.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change looks good -- I think we're just blocked on readthedocs/readthedocs.org#11209
@@ -105,7 +105,7 @@ Why do we collect this information? | |||
- We limit our use of your User Personal Information to the purposes listed in this Privacy Statement. | |||
If we need to use your User Personal Information for other purposes, we will ask your permission first. | |||
You can always see what information we have in your | |||
`user account <https://readthedocs.org/accounts/edit/>`__. | |||
`user account <https://app.readthedocs.org/accounts/edit/>`__. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think most of these probably need some similar solution to the dashboard links, where we have a way to link to a page that asks the user to go to .com or .org. It's probably out of scope for this PR though 🙃
Already noted, but this is blocked on the notification to link users back to the legacy dasboard. With this, there would be no way to easily switch back. |
Seems safe to merge this now 👍 |
We are already migrated on community, so I think we can start pointing people to the new dashboard already.
📚 Documentation preview 📚: https://readthedocs-about--332.org.readthedocs.build/