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

[AP-143] Remove all references to the refresh token from the technical documentation #182

Closed
wants to merge 1 commit into from

Conversation

pauldougan
Copy link
Contributor

@pauldougan pauldougan commented Mar 13, 2024

AP-143

Why

We are deprecating the refresh token and want to ensure that relying party developers do not use it.

What

Remove all reference to the refresh token and the offline_access scope from the technical documentation content:

Technical writer support

Yes

How to review

  • check out the branch
  • run the middleman server locally
  • review modified pages and diagrams
  • ensure overall consistency of the documentation

@pauldougan
Copy link
Contributor Author

pauldougan commented Mar 21, 2024

@philf999 @uche234 (can you confirm on the PR that you are happy with the changes then we can hand this over to the tech writers

@pauldougan pauldougan requested review from uche234 and removed request for JacksonJ2W March 21, 2024 11:17
Copy link
Contributor

@philf999 philf999 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

@uche234
Copy link

uche234 commented Mar 21, 2024

All references to the Refresh token were removed. Looks great.

@ethanmills
Copy link
Member

I don't see any changes to that diagram?

@pauldougan
Copy link
Contributor Author

@ethanmills this is an old draft branch which I need to kill off the actual pr is #185

@pauldougan pauldougan closed this Mar 25, 2024
@pauldougan pauldougan deleted the deprecate-refresh-token branch April 9, 2024 15:43
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.

4 participants