Skip to content

docs: clarify Quickstart POST example (fixes #6949)#6960

Merged
nateprewitt merged 2 commits intopsf:mainfrom
E8Price:docs/quickstart-fix-6949
Mar 3, 2026
Merged

docs: clarify Quickstart POST example (fixes #6949)#6960
nateprewitt merged 2 commits intopsf:mainfrom
E8Price:docs/quickstart-fix-6949

Conversation

@E8Price
Copy link
Contributor

@E8Price E8Price commented Jun 2, 2025

Remove the line that says r1.text == r2.text == True and add a note about X-Amzn-Trace-Id, showing how to compare r1.json()['form'] instead.

@nateprewitt nateprewitt force-pushed the docs/quickstart-fix-6949 branch from d85e1b8 to 9c50ac6 Compare March 3, 2026 19:51
Copy link
Member

@nateprewitt nateprewitt left a comment

Choose a reason for hiding this comment

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

Thanks for the PR, @E8Price. I've made a few small cleanup changes but this looks good. Generally we want to keep PRs focused to a specific change and avoid extra fixes. Those can be broken out into their own PRs, although we generally won't take "nit" PRs unless the meaningfully affect rendering.

@nateprewitt nateprewitt force-pushed the docs/quickstart-fix-6949 branch from 9c50ac6 to 3dc4bd4 Compare March 3, 2026 19:53
@nateprewitt nateprewitt merged commit d568f47 into psf:main Mar 3, 2026
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants