-
Notifications
You must be signed in to change notification settings - Fork 4.5k
[DOCS] Adding GUI docs to database overview #30314
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
Conversation
|
CI Results: |
|
Build Results: |
| 1. Click on **Connections** tab and click **Create connection +** | ||
| 1. Click on Database plugin dropdown and select desired plugin ie. **postgreSQL** | ||
|
|
||
|  |
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.
We should limit screenshots to one per tab to avoid overloading the instructions. Pick whichever of the images sets the best context for the user and make that the top image :)
|
@drivera258 I created #30325 to suggest format/style changes. But this might overlaps with Sarah's suggestions. So, take my style suggestion as a "possible style" and you don't need to accept my suggestions. I'll keep my PR open for now. |
thanks @yhyakuna ! I pulled in some of those changes along with Sarah's suggestions here |
|
@schavis I know you suggested to use an unordered list for the UI steps. But I think it should use a numbered list all the way through.
|
|
@schavis any comment on Yoko's suggestion ? ^ |
|
I agree the steps should be numbered. The reason it's unordered is because we use a partial for the first part of the steps to enforce consistency in how we present those instructions and the platform resets the numbering after the partial 😞 |
* test doc * spacing * update index & scrns * fixes * adding static info * fix wording * updates * applying suggested changes * condense tabs * removing unused files * separate tabs

Description
Vercel link to overview page