-
Notifications
You must be signed in to change notification settings - Fork 620
docs: improve confluence docs #5049
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
khaliqgant
left a comment
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.
This kind of conflicts with #5056. Can you combine the changes to confluence with that one so we don't have any conflicts and remove it from this PR?
🤖 My Senior Dev — Analysis Complete👤 For @hassan254-prog📁 Expert in View your contributor analytics → 📊 5 files reviewed
🚀 Open Interactive Review →The full interface unlocks features not available in GitHub:
💬 Chat here: 📖 View all 12 personas & slash commandsYou can interact with me by mentioning In PR comments or on any line of code:
Slash commands:
AI Personas (mention to get their perspective):
For the best experience, view this PR on myseniordev.com — includes AI chat, file annotations, and interactive reviews. |
## Describe the problem and your solution - Improve confluence docs and other minor cleanups. <!-- Issue ticket number and link (if applicable) --> <!-- Testing instructions (skip if just adding/editing providers) --> <!-- Summary by @propel-code-bot --> --- **Refined Confluence integration docs and added OAuth token guidance** Restructures the Confluence integration documentation into a tabbed layout that emphasizes quickstart steps, OAuth setup, and refreshed guidance on `connectionConfig.cloudId`, including the new `read:page:confluence` scope and a clearer explanation of multi-site domain handling. Adds a Facebook OAuth note about 60-day token expiry, updates the companion setup guide, and replaces the `packages/webapp/public/images/template-logos/apollo.svg` asset with a higher-contrast yellow version. <details> <summary><strong>Key Changes</strong></summary> • Overhauled `docs/integrations/all/confluence.mdx` to a tabbed quickstart with updated sample requests and expanded `cloudId`/domain selection guidance. • Simplified `docs/api-integrations/confluence/how-to-register-your-own-confluence-api-oauth-app.mdx` by removing the prerequisites table and clarifying scope selection and multi-site behavior. • Documented Facebook token refresh limitations in `docs/api-integrations/facebook/how-to-register-your-own-facebook-api-oauth-app.mdx` and `docs/integrations/all/facebook.mdx`. • Swapped `packages/webapp/public/images/template-logos/apollo.svg` for a yellow variant to improve visibility. </details> <details> <summary><strong>Affected Areas</strong></summary> • docs/integrations/all/confluence.mdx • docs/api-integrations/confluence/how-to-register-your-own-confluence-api-oauth-app.mdx • docs/api-integrations/facebook/how-to-register-your-own-facebook-api-oauth-app.mdx • docs/integrations/all/facebook.mdx • packages/webapp/public/images/template-logos/apollo.svg </details> --- *This summary was automatically generated by @propel-code-bot*
Describe the problem and your solution
Refined Confluence integration docs and added OAuth token guidance
Restructures the Confluence integration documentation into a tabbed layout that emphasizes quickstart steps, OAuth setup, and refreshed guidance on
connectionConfig.cloudId, including the newread:page:confluencescope and a clearer explanation of multi-site domain handling. Adds a Facebook OAuth note about 60-day token expiry, updates the companion setup guide, and replaces thepackages/webapp/public/images/template-logos/apollo.svgasset with a higher-contrast yellow version.Key Changes
• Overhauled
docs/integrations/all/confluence.mdxto a tabbed quickstart with updated sample requests and expandedcloudId/domain selection guidance.• Simplified
docs/api-integrations/confluence/how-to-register-your-own-confluence-api-oauth-app.mdxby removing the prerequisites table and clarifying scope selection and multi-site behavior.• Documented Facebook token refresh limitations in
docs/api-integrations/facebook/how-to-register-your-own-facebook-api-oauth-app.mdxanddocs/integrations/all/facebook.mdx.• Swapped
packages/webapp/public/images/template-logos/apollo.svgfor a yellow variant to improve visibility.Affected Areas
• docs/integrations/all/confluence.mdx
• docs/api-integrations/confluence/how-to-register-your-own-confluence-api-oauth-app.mdx
• docs/api-integrations/facebook/how-to-register-your-own-facebook-api-oauth-app.mdx
• docs/integrations/all/facebook.mdx
• packages/webapp/public/images/template-logos/apollo.svg
This summary was automatically generated by @propel-code-bot