Skip to content

Update Postman collection for 12-9-24 launch#31

Merged
nickfryar merged 6 commits intomainfrom
12_9_24-launch-postman-update
Dec 10, 2024
Merged

Update Postman collection for 12-9-24 launch#31
nickfryar merged 6 commits intomainfrom
12_9_24-launch-postman-update

Conversation

@nickfryar
Copy link
Copy Markdown
Member

Updated the Postman collection with latest features

image

Comment thread postman/threads-api.postman_collection.json Outdated
Comment thread postman/threads-api.postman_collection.json Outdated
Comment thread postman/threads-api.postman_collection.json Outdated
@@ -2164,24 +2465,24 @@
{
"key": "fields",
"value": "{{fields_threads}}",
Copy link
Copy Markdown
Contributor

@anujs3 anujs3 Dec 10, 2024

Choose a reason for hiding this comment

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

Should this be {{fields_replies}}?

image

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Weird, I checked the collection and it seems to be correct

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Look at the example.

image

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Ah I see, fixed

Comment thread postman/threads-api.postman_collection.json
Comment on lines +2732 to +2737
{
"key": "access_token",
"value": "{{app_access_token}}",
"description": "This is your app access token received via the GET /oauth/access_token endpoint. See the Authorization folder for details.",
"disabled": true
}
Copy link
Copy Markdown
Contributor

@anujs3 anujs3 Dec 10, 2024

Choose a reason for hiding this comment

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

It looks like there are two keys for access_token. Let's remove this one (the second instance) and keep the other one (the first instance).

image

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

These are different. App access tokens can be either the combination of the id and secret (top) or an encrypted token (bottom; via the GET /oauth/access_token endpoint)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ah ok got it. So does that mean you can use either one for oEmbed?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yup either one works

Comment on lines +2767 to +2772
{
"key": "access_token",
"value": "{{app_access_token}}",
"description": "This is your app access token received via the GET /oauth/access_token endpoint. See the Authorization folder for details.",
"disabled": true
}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Same as comment above. Let's remove the duplicate access_token.

image

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

See above answer

Comment thread postman/threads-api.postman_collection.json Outdated
Comment thread postman/threads-api.postman_collection.json Outdated
Comment thread postman/threads-api.postman_collection.json Outdated
@nickfryar nickfryar merged commit 7478840 into main Dec 10, 2024
@anujs3 anujs3 deleted the 12_9_24-launch-postman-update branch March 11, 2025 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants