Skip to content

What to do when credentials have updated/changed #4018

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

Merged

Conversation

daldeano-segment
Copy link
Contributor

Proposed changes

Updating Getting Started Section with the following Information:

Note: It's important to note that when a connected account changes or updates their credentials, a new Facebook Lead Ads source will have to be created instead of solely reauthorizing the account as this is how FB has decided to operate. Reauthorization implies that a new token gets generated, but the token is not the sole link between Segment and Facebook. A Segment app is installed onto an FB Account and that will be the sprouting point from where tokens are generated.

The reason we can't disconnect and reconnect the CRM on Facebook with reauthorization is because the creation of a new token will mean to invalidate the rest of the tokens attached to the same Facebook account. For example, let's imagine you have a Facebook Account with three accounts: Account A, Account B, and Account C. You connect Segment to all of them and then generate tokens respectively. Later on, you decide for whatever reason to reauthorize the "Account A" page, the other tokens will be invalidated, and the pages will stop working.

Customer

Merge timing

Related issues (optional)

Copy link
Contributor

@charholt charholt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks great!

@daldeano-segment daldeano-segment marked this pull request as ready for review January 18, 2023 21:07
@daldeano-segment daldeano-segment requested a review from a team as a code owner January 18, 2023 21:07
@daldeano-segment daldeano-segment requested review from rchinn1 and removed request for a team January 18, 2023 21:07
@daldeano-segment daldeano-segment merged commit 19558a2 into develop Jan 18, 2023
@daldeano-segment daldeano-segment deleted the daldeano-segment-facebook-lead-ads-patch-1-1 branch January 18, 2023 21:09
@github-actions
Copy link
Contributor

Thank you for your contribution! Your pull request is merged, but may take a day or two to appear on the site.

@markzegarelli
Copy link
Contributor

Hi @daldeano-segment, thanks for the PR!

In future PRs though, please wait for approval from a Docs team member before you merge. There are a few fixes we need to add here, that our grammar linter caught.

I will fix those in a separate PR, but you can see the warnings on the Files Changed tab.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants