try to retrieve page token from page details via user token #114
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
pivnicek
approved these changes
Sep 15, 2023
Comment on lines
32
to
33
| (runtime/log-strings "Could not find page access token for" id ". Token from the configuration will be used instead.") | ||
| (runtime/log-strings "Using page access token to retrieve data for" id)) |
There was a problem hiding this comment.
I know it's not part of the change, but isn't the second log message here a lie? the first message says that it couldn't find the page access token, and the second one says that it's using the page access token (that it couldn't find?
There was a problem hiding this comment.
good question, actually this is if syntax of clojure, first expression in the list is if the condition passes, the second expression in the list is if the condition fails (then expression) so it is like if-then-else but the else keyword is missing here :)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
fixes https://keboola.atlassian.net/browse/ST-872
Tato uprava fixuje ex facebook, kde po bc change na fb api prestali vracat fb pages v requeste pre zoznam accountov (
me/accounts) z ktoreho si fb ex nacital page access token. To je reportovane ako bug ale fb sa moc nema k tomu aby to fixol vid https://developers.facebook.com/support/bugs/602264498523167/Po novom to extractor okrem me/accounts skusa ako fallback aj cez
/PAGE-ID?fields=access_token(https://developers.facebook.com/docs/pages/access-tokens), ako je poradene v tom bug reporte:Test