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

HAI-1884 Identify user coming from invitation link #374

Merged
merged 2 commits into from
Sep 25, 2023

Conversation

markohaarni
Copy link
Contributor

Description

When user comes to Haitaton from invitation link, call identify endpoint with token included in the link. After identifying user, show success notification and redirect user to front page. If user is not signed in before coming to Haitaton, redirect user to login and after login, do the identifying.

Jira Issue: https://helsinkisolutionoffice.atlassian.net/browse/HAI-1884

Type of change

  • Bug fix
  • New feature
  • Other

Instructions for testing

Please describe tests how this change or new feature can be tested.

Checklist:

  • I have written new tests (if applicable)
  • I have ran the tests myself (if applicable)
  • I have made necessary changes to the documentation, link to confluence
    or other location:

Other relevant info

Please describe here if there is e.g. some requirements for this change or
other info that the tester/user needs to know.

When user comes to Haitaton from invitation link, call identify
endpoint with token included in the link. After identifying user,
show success notification and redirect user to front page.
If user is not signed in before coming to Haitaton, redirect user
to login and after login, do the identifying.
…tification (#375)

Added hanke name and tunnus to notification that is shown after
successfully identifying user.
@markohaarni markohaarni merged commit 0e2c672 into dev Sep 25, 2023
1 check passed
@markohaarni markohaarni deleted the HAI-1884/identify-user branch September 25, 2023 12:44
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