Skip to content
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

fixed tiny typo to add an R #181

Merged
merged 1 commit into from
Mar 25, 2024
Merged

Conversation

ImogenCraigmile
Copy link
Contributor

Why

Small change to fix a typo and therefore improve accuracy

What

In secure your authorisation request parameters with JWT, it refers to YOU_REQUEST_OBJECT where it should be YOUR_REQUEST_OBJECT

Technical writer support

Yes - needs a sanity check that I've updated it in the right place

How to review

Look at the files changed and check how it looks when ran locally

@ImogenCraigmile ImogenCraigmile requested review from a team as code owners March 11, 2024 09:47
Copy link
Contributor

@seaemsi seaemsi left a comment

Choose a reason for hiding this comment

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

Looks good to me!

Copy link
Contributor

@sdurnberger sdurnberger left a comment

Choose a reason for hiding this comment

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

Thanks Imogen!

@pauldougan
Copy link
Contributor

@huwd can you help us merge this simple typo change to the tech docs pls

@huwd huwd merged commit 010ec3c into main Mar 25, 2024
2 checks passed
@huwd huwd deleted the fix_typo_in_request_params_JWT_docs branch March 25, 2024 12:35
@huwd
Copy link
Contributor

huwd commented Mar 25, 2024

Yep, right that's it gonna get an adjustment to code-owners up now.
This is silly.

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.

5 participants