-
Notifications
You must be signed in to change notification settings - Fork 40
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
Update facebook.mdx #68
Conversation
Added importance of changing App Mode to Live - doesn't work otherwise (although appears to be functioning) Added comment about advanced permissions not being required for personal use only.
@FoamToaster is attempting to deploy a commit to the Listinai Team on Vercel. A member of the Team first needs to authorize it. |
WalkthroughThe update clarifies the Facebook integration setup instructions in the documentation. It explains that advanced permissions are not required for users with a personal Postiz installation, and it adds a step instructing users to change the App Mode from "Development" to "Live" to ensure posts are visible to others. Additionally, it introduces a new Step 9 for copying the App ID and App Secret. Changes
Sequence Diagram(s)sequenceDiagram
participant U as User
participant D as Facebook Integration Doc
participant F as Facebook App
U->>D: Read integration instructions
D-->>U: Display note on advanced permissions
U->>D: Check requirements for personal installation
D-->>U: Instruct to switch App Mode (Development -> Live)
U->>F: Change App Mode to Live
D-->>U: Present new step to copy App ID & App Secret
U->>F: Copy credentials for continued setup
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
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 (3)
pages/providers/facebook.mdx (3)
59-60
: Clarification on Advanced PermissionsThe added note clearly informs users that if their Postiz installation is for personal use only, advanced permissions are not required. This helps prevent any confusion regarding the necessity of these permissions. Consider enhancing the visibility of this note—perhaps by applying additional formatting (such as bolding or wrapping it in a callout) to ensure that users do not overlook this important detail.
62-63
: Critical App Mode UpdateThe new step instructs users to change the App Mode from "Development" to "Live," which is vital for ensuring that posts made via the API are visible to all users rather than just the creator. The explanation is clear and adequately emphasizes the consequences of not making this change.
64-64
: Introduction of Step 9 for Credential ConfigurationThe addition of Step 9, which directs users to copy their App ID and App Secret, aligns well with the overall Facebook integration setup. For added clarity—and to promote security best practices—you might consider adding a brief note reminding users to keep their credentials secure.
Added importance of changing App Mode to Live - doesn't work otherwise (although appears to be functioning) Added comment about advanced permissions not being required for personal use only.
Summary by CodeRabbit