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

Use the same extraParams during renewal as during initial sign in #1525

Closed
wants to merge 2 commits into from

Conversation

MelvinVermeer
Copy link

This PR Implements the proposed solution from this issue #1524

For my convenience I chose to use the sharedTransactionStorage, but this could be any existing storage, or a new one.

Extra bits

This PR also includes a non-breaking package upgrade of rollup-plugin-typescript2 because this was needed to make yarn build work.

Melvin Vermeer added 2 commits July 4, 2024 15:51
@jaredperreault-okta
Copy link
Contributor

Internal Ref: OKTA-750886

@jaredperreault-okta
Copy link
Contributor

@MelvinVermeer This was resolved by #1529 and just released to npm in 7.7.1

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