-
Notifications
You must be signed in to change notification settings - Fork 4.8k
embedded quick start and rearranging a few things #67613
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
👋 Greetings, Airbyte Team Member!Here are some helpful tips and reminders for your convenience. Helpful Resources
PR Slash CommandsAirbyte Maintainers (that's you!) can execute the following slash commands on your PR:
|
- Move tutorial document paths to tutorials/ subdirectory - Add new quickstart page to sidebar - Add template-tags to sidebar that was previously missing - Organize tutorials under a collapsible Tutorials category
- Fix README.md link to point to tutorials/prerequisites-setup.md - Update image paths in tutorial markdown files to use ../assets/ instead of ./assets/ - Ensures all links and images resolve correctly after file reorganization
Deploy preview for airbyte-docs ready! ✅ Preview Built with commit eb02400. |
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.
LGTM! I left a few suggestions, none of which are blocking. Just things that popped into my head as I read.
The other linter also had a few suggestions. Good to accept but again, not blocking.
What
Based on some customer feedback that our setup guides were a bit difficult to follow and they just wanted a one page step-by-step.
This does depend on https://github.com/airbytehq/sonar/pull/1049 being merged, which moves the Applications Token endpoint to the public Sonar API (another request from that customer!)