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 redirecting legacy users to the legacy app #247

Merged
merged 5 commits into from
Oct 3, 2018

Conversation

toolness
Copy link
Collaborator

@toolness toolness commented Oct 3, 2018

This is the complement to JustFixNYC/tenants#203, and fixes #78 by adding support for auto-logging-in legacy users to the legacy tenants' app and redirecting them there.

Currently it works like this:

  • When a user logs in to the new app, if they're not a "first-class" user (i.e. someone who registered on the new app), we'll check to see if they are a legacy user. If they are, we create a LegacyUserInfo record for them, migrate over their first and last name, and set a "prefers legacy app" flag for them (this flag can be changed through the Django admin interface, if we ever want to allow them to log into the new app without being forcibly redirected to the legacy one).

  • Once login is successful, the front-end checks to see if the user has the "prefers legacy app" flag set. If they do, they are redirected to the legacy app by creating an autologin document in the legacy app's MongoDB instance and passing the key in the URL.

To do

  • Add tests.
  • Make the front-end redirect the user to the legacy app after login if needed.

@toolness toolness added this to the The First Milestone milestone Oct 3, 2018
@toolness toolness changed the title [WIP] Add legacy redirect support [WIP] Add support for redirecting legacy users to the legacy app Oct 3, 2018
@toolness toolness changed the title [WIP] Add support for redirecting legacy users to the legacy app Add support for redirecting legacy users to the legacy app Oct 3, 2018
@toolness toolness merged commit 0a7b16e into master Oct 3, 2018
@toolness toolness deleted the legacy-redirect branch October 3, 2018 14:37
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.

Determine a user account migration/integration strategy
1 participant