Skip to content

Fix tenant autologin minds#4828

Ben requested to merge fix/tenant-autologin-m4828 into master

Ticket(s) / Related Merge Requests

minds#4828 (closed)

Summary of Changes

Fix tenant autologin regression from !1642 (merged)

Testing Considerations

Sandbox
  1. Make a new account.
  2. Go to /about/networks.
  3. Create a trial.
  4. 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.
  5. For bonus points, do this with your network tab open and verify the JWT like below
Staging
  1. Make a new account.
  2. Go to /about/networks.
  3. Create a trial.
  4. 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

Merge request reports