-
Notifications
You must be signed in to change notification settings - Fork 4.9k
docs(source-facebook-pages): improve OAuth permissions documentation #70275
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
base: master
Are you sure you want to change the base?
docs(source-facebook-pages): improve OAuth permissions documentation #70275
Conversation
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:
|
|
@ian-at-airbyte I am an AI (Devin) and have proposed documentation updates based on recent changes to the source-facebook-pages connector (specifically commit a4bfeec). The changes improve the OAuth permissions documentation by:
You can merge this PR, modify it, or close it if you disagree with the changes. Note: I was unable to verify the Docusaurus build locally due to pre-existing environment issues with the postman-code-generators package. The documentation change is minimal and unlikely to cause broken links. |
|
Deploy preview for airbyte-docs ready! ✅ Preview Built with commit 1948dc3. |
What
Improves the OAuth permissions documentation for the Facebook Pages source connector by:
read_insightsto the required permissions list (needed for Page Insights and Post Insights streams)This change was prompted by reviewing commit a4bfeec which added the OAuth App section.
How
Reorganized the permissions list in
docs/integrations/sources/facebook-pages.mdto distinguish between:pages_read_engagement,pages_read_user_content,pages_show_list,read_insightspages_manage_ads,pages_manage_metadata,catalog_managementReview guide
docs/integrations/sources/facebook-pages.md- Verify the permission categorization is accurate based on Facebook Graph API requirementsUser Impact
Users creating their own OAuth App will have clearer guidance on which permissions are required for basic functionality vs. which are needed for advanced features.
Can this PR be safely reverted and rolled back?
Link to Devin run: https://app.devin.ai/sessions/50dcafdaac264bad82044c5b3dad8b0d
Requested by: unknown ()
Note: I was unable to verify the Docusaurus build locally due to pre-existing environment issues with the postman-code-generators package. The documentation change is minimal and unlikely to cause broken links.