Fix tenant autologin minds#4828
Ticket(s) / Related Merge Requests
Summary of Changes
Fix tenant autologin regression from !1642 (merged)
Testing Considerations
Sandbox
- Make a new account.
- Go to
/about/networks
. - Create a trial.
- You should be redirected to
/api/v3/multi-tenant/auto-login/login
with a new tenant subdomain, but the page WILL be a 404. - check the JWT in the response and verify the user_guid is correct. - For bonus points, do this with your network tab open and verify the JWT like below
Staging
- Make a new account.
- Go to
/about/networks
. - Create a trial.
- You should be redirected to
/api/v3/multi-tenant/auto-login/login
and be automatically logged into your tenant.
All envs
-
Test navigation to existing networks works at /network on non-tenant domain.
Deployment Considerations
Standard deployment.
Regression Scope
Tenant auto-login.
Platform Affected (web, mobile, etc)
Web.
Developer Testing Completed
Manual, unit (phpspec).
Screenshots / Screen Recording
Sandbox
Network trial creation autologin
simplescreenrecorder-2024-04-26_13.56.09
/network page autologin
simplescreenrecorder-2024-04-26_13.55.19
Does this impact
-
Localization -
Dark/light mode -
Guest mode
Definition of Done Checklist
-
The Acceptance Criteria has been met -
Code is tested: Testing includes unit/spec, E2E/automated and manual testing -
Merge requests description has been filled out
Edited by Ben