-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Update onboarding #8504
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
Update onboarding #8504
Conversation
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 is much better UX! 💯
You may need to do a grep in other repositories (in particular -corporate) because there may be views that still depend on this class. Also, you will need to double check that the onboard_import.html
file on -corporate does not require a change.
That said, I think we can't merge this PR without checking those things first.
+1 on transfering this |
Good point, will check the corporate site now.
|
9bb15df
to
30126b3
Compare
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.
Yay deleting code :)
Migrated the tutorial template to our GitHub organization https://github.com/readthedocs/tutorial-template/ and replaced another two links on the homepage to point to the Tutorial. Merging! |
Addresses #8410 by
This is how it looks like now:
This also removes the demo code.