-
-
Notifications
You must be signed in to change notification settings - Fork 824
chore updated docs #165
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
chore updated docs #165
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
WalkthroughThe updates include documentation and configuration changes: the README screenshot for the "Research Agent" was replaced and repositioned, the backend environment variable example file was cleaned of obsolete LLM configuration comments, and both Docker and manual installation guides were reorganized to consolidate environment variable documentation, removing deprecated LLM variables and adding new audio and Google OAuth variables. Changes
Poem
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
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.
Actionable comments posted: 0
🧹 Nitpick comments (1)
README.md (1)
2-2: Add descriptive alt text and optimize image attributes
The new header image should include meaningful alt text for accessibility and consider specifying dimensions or styling classes to maintain consistent layout across viewports.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (4)
README.md(1 hunks)surfsense_backend/.env.example(0 hunks)surfsense_web/content/docs/docker-installation.mdx(1 hunks)surfsense_web/content/docs/manual-installation.mdx(1 hunks)
💤 Files with no reviewable changes (1)
- surfsense_backend/.env.example
🔇 Additional comments (5)
README.md (1)
150-152: Repositioned "Research Agent" section
The updated heading and screenshot placement look aligned with the "Search Spaces" flow. Confirm the caption and flow remain clear to readers.surfsense_web/content/docs/manual-installation.mdx (2)
57-60: Integrate Google OAuth variables into main env list
The addition ofAUTH_TYPE,GOOGLE_OAUTH_CLIENT_ID, andGOOGLE_OAUTH_CLIENT_SECRETis clear and concise. Please verify this ordering and naming exactly match the backend.env.example.
63-69: Document audio service and Firecrawl API variables
IntroducingTTS_SERVICE,STT_SERVICEand their keys alongsideFIRECRAWL_API_KEYenhances clarity. Double-check there are no duplicated entries elsewhere in the doc or examples.surfsense_web/content/docs/docker-installation.mdx (2)
87-90: Align OAuth config in Docker env table
The Docker guide now includesAUTH_TYPE,GOOGLE_OAUTH_CLIENT_ID, andGOOGLE_OAUTH_CLIENT_SECRET. Ensure these entries and any default/example values match the manual installation docs and backend.env.example.
93-99: Update audio service and Firecrawl env vars
The TTS/STT entries and repositionedFIRECRAWL_API_KEYalign with backend changes. Please confirm there are no remaining references to deprecated LiteLLM variables and no duplicates across the three env docs.
chore updated docs
chore updated docs
Description
chore updated docs
API Changes
Types of changes
Testing
Checklist:
Summary by CodeRabbit