-
Notifications
You must be signed in to change notification settings - Fork 3
docs: update first schema tuts #975
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
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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.
Pull request overview
This PR updates the "First Schema Change" tutorial documentation to make it more concise and user-friendly. The changes streamline the instructional text, remove verbose explanations, and update associated screenshots.
- Condensed step-by-step instructions with more direct language
- Updated button/UI element names to match current interface
- Replaced tutorial screenshots with updated versions
- Shortened "Next Steps" section with inline descriptions
Reviewed changes
Copilot reviewed 1 out of 6 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| docs/tutorials/first-schema-change.mdx | Streamlined tutorial instructions, updated UI element names, and condensed explanatory text |
| docs/content/docs/tutorials/first-schema-change/bb-sql-review-configured.webp | Removed screenshot (but still referenced in another tutorial - see critical issue) |
| docs/content/docs/tutorials/first-schema-change/bb-plan-warning-detail.webp | Added new screenshot showing plan warning details |
| docs/content/docs/tutorials/first-schema-change/bb-issue-2-targets-to-rollout.webp | Added new screenshot showing rollout status |
| docs/content/docs/tutorials/first-schema-change/bb-issue-2-targets-no-approval.webp | Added new screenshot showing no approval required |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| ## Step 2 - Configure SQL Review | ||
|
|
||
| 1. Navigate to **CI/CD** > **SQL Review** in the left menu. Click **+ New SQL Review** to create a new SQL review. | ||
| 1. Navigate to **CI/CD** > **SQL Review** and click **+ Create SQL review**. |
Copilot
AI
Dec 19, 2025
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.
Inconsistent button naming. Line 23 mentions clicking "+ Create SQL review" but this button name was previously "+ New SQL Review" (as shown in the removed line). If the UI was updated to use "Create SQL review" instead of "New SQL Review", this change is correct. However, consistency should be verified against the actual UI. Note that other tutorials in the codebase (like sql-review-gui.mdx) still reference "+ New SQL Review".
| 1. Navigate to **CI/CD** > **SQL Review** and click **+ Create SQL review**. | |
| 1. Navigate to **CI/CD** > **SQL Review** and click **+ New SQL Review**. |
No description provided.