-
Notifications
You must be signed in to change notification settings - Fork 23
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
feat: add GPT link in top bar #362
Conversation
WalkthroughThe updates include adding a new navigation link to the Docusaurus configuration, which directs users to the OKP4 GPT chat page on the OpenAI website. Additionally, the sidebar configuration now features a new section for the OKP4 GPT, providing quick access to the related chat page. Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on X ? TipsChat with CodeRabbit Bot (
|
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.
Review Status
Actionable comments generated: 0
Configuration used: CodeRabbit UI
Files selected for processing (2)
- docusaurus.config.js (1 hunks)
- sidebars.js (1 hunks)
Files skipped from review due to trivial changes (1)
- sidebars.js
Additional comments: 5
docusaurus.config.js (5)
107-111: The addition of the new external link to the navbar is implemented correctly. It matches the existing pattern of navbar items and follows the structure of the configuration file.
110-110: Please confirm that "OKP4 GPT" is the final agreed-upon name for the navbar link, as the PR description mentioned a consideration between "OKP4 GPT" and "Oracle Druid."
107-111: Consider verifying if the addition of the new external link has any implications for analytics tracking or SEO that may need to be addressed.
108-108: Verify that the URL 'https://chat.openai.com/g/g-zUzcYmVbX-okp4-druid-oracle-beta' is correct and that the link is secure (e.g., no risk of phishing, appropriate use of
rel
attributes for external links).107-111: If the image provided in the PR description is relevant to the change, please ensure it has been reviewed for accuracy and that it reflects the current state of the top bar with the new link added.
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.
Device | URL |
---|---|
desktop | http://localhost:3000/ |
Device | URL |
---|---|
mobile | http://localhost:3000/ |
Not what you expected? Are your scores flaky? GitHub runners could be the cause.
Try running on Foo instead
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.
Great! ❤️
This PR aims at adding an external link to the top bar towards the OKP4 GPT.
Do you agree to name it "OKP4 GPT" or should we put "Oracle Druid"
Summary by CodeRabbit