-
Notifications
You must be signed in to change notification settings - Fork 5k
docs(source-facebook-pages): improve grammar and fix misleading reference in documentation #70243
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
docs(source-facebook-pages): improve grammar and fix misleading reference in documentation #70243
Conversation
…ence in documentation Co-Authored-By: unknown <>
Original prompt from API User |
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
👋 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:
|
|
Hi @ian-at-airbyte, I am an AI (Devin) and have proposed documentation updates based on a review of commit e21c11e, which updated the Facebook Pages connector to v24 of the Facebook Graph API. After reviewing the connector source code and the official Facebook Graph API documentation, I identified several improvements to the user documentation:
You can merge this PR, modify it, or close it if you disagree with the changes. This PR was requested by unknown (). |
| 5. Fill in Page ID (if you have a page URL such as `https://www.facebook.com/Test-1111111111`, the ID would be`Test-1111111111`) | ||
| 5. Fill in Page ID (if you have a page URL such as `https://www.facebook.com/Test-1111111111`, the ID would be `Test-1111111111`) | ||
|
|
||
| ### For Airbyte OSS: |
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.
[markdownlint-fix] reported by reviewdog 🐶
| ### For Airbyte OSS: | |
| ### For Airbyte OSS |
|
Deploy preview for airbyte-docs ready! ✅ Preview Built with commit 677d0b9. |
This PR consolidates changes from three separate PRs (#70243, #70275, #70276) into a single cohesive update: Grammar and formatting improvements: - Added missing articles (a Facebook Developer Account, a Facebook App) - Improved phrasing (these instructions, the following permissions) - Added missing colon after introductory sentence - Fixed missing space before backtick in code formatting - Removed trailing colon from heading per markdownlint Reference and content fixes: - Fixed misleading 'Setup Guide below' to 'Setup Guide above' - Made rate limiting guidance more specific (Long-Lived Page Token) OAuth permissions alignment: - Aligned OAuth App permissions with Step 3 setup guide - Added read_insights (required for Page Insights and Post Insights streams) - Removed unnecessary permissions (pages_manage_ads, pages_manage_metadata, catalog_management) - Fixed typo 'The Facebook Pages' to 'the Facebook Pages' Co-Authored-By: [email protected] <[email protected]>
What
Improves the Facebook Pages connector documentation by fixing grammar issues and correcting a misleading reference that could confuse users.
This PR was created by an AI (Devin) as part of an automated documentation review following commit e21c11e.
Link to Devin run: https://app.devin.ai/sessions/b160b1ba089d4a33a4cd3af256509762
How
Review guide
docs/integrations/sources/facebook-pages.md- All changes are in this single fileKey changes to verify:
User Impact
Users will have clearer, more grammatically correct documentation. The fix to the "Setup Guide below" reference removes potential confusion when users are looking for rate limiting guidance.
Can this PR be safely reverted and rolled back?