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

fix: added support for redirectUrl for netcetera flow #636

Merged
merged 1 commit into from
Sep 23, 2024

Conversation

ArushKapoorJuspay
Copy link
Contributor

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates
  • Documentation
  • CI/CD

Description

Added Support for RedirectUrl for Netcetera Flow with SDK Button

How did you test it?

Test both frictionless and challenge flow for netcetera via SDK button. (The redirected url should have client_secret and status appended to it)

Checklist

  • I ran npm run re:build
  • I reviewed submitted code
  • I added unit tests for my changes where possible

Copy link

Review changes with SemanticDiff.

@ArushKapoorJuspay ArushKapoorJuspay self-assigned this Sep 23, 2024
Copy link
Collaborator

@PritishBudhiraja PritishBudhiraja left a comment

Choose a reason for hiding this comment

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

Discussed these changes with @ArushKapoorJuspay

@PritishBudhiraja PritishBudhiraja merged commit 205054f into main Sep 23, 2024
3 checks passed
@PritishBudhiraja PritishBudhiraja deleted the fix/netcetera-return-url branch September 23, 2024 13:07
@github-actions github-actions bot added the Closed Label will be automatically added when the PR will get merged to main label Sep 23, 2024
akash-c-k pushed a commit that referenced this pull request Sep 23, 2024
## [0.88.4](v0.88.3...v0.88.4) (2024-09-23)

### Bug Fixes

* added support for redirectUrl for netcetera flow ([#636](#636)) ([205054f](205054f))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Closed Label will be automatically added when the PR will get merged to main
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants