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.

Sign up for GitHub

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

Jump to bottom

fix: saved idx transaction should be available after terminal response #1185

Closed
wants to merge 7 commits into from
Closed

fix: saved idx transaction should be available after terminal response #1185

wants to merge 7 commits into from

Conversation

aarongranick-okta
Copy link
Contributor

@aarongranick-okta aarongranick-okta commented Apr 15, 2022 •
edited
Loading

  • does not clear IDX response when reaching a terminal state (transaction meta is still cleared)
  • stateHandle and interactionHandle (if available) are stored along with the last response
  • access to the last IDX response can be gated by a matching stateHandle or interactionHandle, if provided by the caller.

This fix maintains compatibility with some stateHandle-based flows which involve executing the "cancel" action after reaching a terminal response.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jaredperreault-okta jaredperreault-okta jaredperreault-okta approved these changes

@shuowu shuowu shuowu approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@aarongranick-okta @shuowu @jaredperreault-okta @eng-services-aperture-github-bot-okta