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

Add support for additional params in auth url #292

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

mpgxvii
Copy link
Member

@mpgxvii mpgxvii commented Jan 16, 2025

  • Add support for additional params in the AuthorizationPage to allow redirecting directly to Fitbit auth and allow redirecting to a custom URL after authorization is complete (using redirect=true and redirect_uri=URL-HERE
  • Also added improvements to the AuthorzationCompletePage
  • Needed by the SEP portal (since user will self authorise and need to be redirected back to the SEP portal)

@yatharthranjan
Copy link
Member

Just reported by callum, if there is an error when linking fitbit (eg - user has already connected fitbit ), then the error is still showin in rest-sources-auth UI. Ideally, it should redirect directly back to the SEP UI if redirect=true is set in the params, and should include the error in the url params.

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