-
Notifications
You must be signed in to change notification settings - Fork 29
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
Consolidate onboarding, connect to Studio, settings & walkthrough #3434
Comments
Some next steps:
|
Do we have an idea on what we want this new section to consist of? Did we want something simple like: Or would there be more to it?
When we say "reminder", are we just think of adding an extra sentence with a doc link or did we want to add more content like our dvclive framework tabs? |
Note from retro: Setup should have a plots section that will help a user to get started with plots, show some snippets they can add to their |
Here is the "Create an env" flow in the Jupyter extension: (fails since I picked to install all the requirements, but idea behind create an env is clear): Screen.Recording.2023-05-11.at.3.27.39.PM.mov |
Reviewed the remaining steps in this issue, considering time, priority, etc., and put together some next steps.
These tasks seem straightforward and should be quick to implement. I'll get started on them in the order above ^
These tasks will take some time and discussion, but I think they are of higher priority in comparison to the rest. Will start these after the smaller tasks mentioned above.
I think we should move the "plots section" task along with the "dvc installation" task to their own issues to make discussion on more concrete next steps easier (with over 10 tasks done already, this issue has gotten quite large 📜 ). |
@julieg18 this plan sounds good to me. |
Working on the "Get Started" section and thought about some solutions to avoid circular references and to further consolidate our Setup and walkthrough. 1. Add a more clear separation between the two pagesKeep our "Get Started" walkthrough focused solely on walking through the extension features and the Setup would be focused on getting the user started with DVC installation. To do this:
2. Move Walkthrough into SetupDelete our "Get Started" walkthrough, showing a walkthrough inside of the Setup "Get Started" section. To do this:
Personally, I like the first option more since we don't need to redesign/rewrite our walkthrough, but if we want our walkthrough simplified anyway and everything within "Setup", the second option would be the better way to go. Or maybe someone else has a better idea in mind :) I'll start working on the first option for now though! |
Yes, I also think the first option is better and it makes sense to me to try to make it more and more like a features overview. |
There are a lot of small disparate screens related to onboarding (Setp webview) and connecting to Studio (Connect webview). There are also buttons associated with these screens in the sidebar. We also have a separate walkthrough (uses VS Code API) which users will see on first-install. The purpose of this ticket is to discuss and then implement a consolidation of this information into a single place.
My idea for an initial implementation involves reusing the layout from the plots webview. Namely this (without the ribbon):
To begin with, we can start by moving the existing sections within this layout. We can also add a new section that links to the walkthrough.
During that process, we can discuss the next steps.
Please LMK your thoughts on
Note: We need to drop the code that automatically opens the walkthrough on first-install.
Related to #3331
Tasks
All Sections
setup/components/App.test.tsx
(Add completion status to setup sections #3823 (comment))DVC Section
LATEST_TESTED_VERSION
to DVC env details, removing or simplifying Toast that is currently used- [ ] Improve our "DVC unavailable" section, making recommended steps to install DVC more clear (#3434 (comment))Replaced by #3935Get Started Section
Experiments Section
Plots Section
- [ ] Add plots section ( #3434 (comment) )Replaced by #3934The text was updated successfully, but these errors were encountered: