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

Remove tenant when tenant-resolve-error triggered #17173

Merged
merged 1 commit into from
Jul 24, 2023
Merged

Conversation

mahmut-gundogdu
Copy link
Contributor

Description

Resolves #volo/14735 (write the related issue number if available)

TODO: Describe what this PR has changed, add screenshot or animated GIF if available, write if it is a breaking change, and how to fix the breaking changes for existing applications if so.

Checklist

  • I fully tested it as developer / designer and created unit / integration tests
  • I documented it (or no need to document or I will create a separate documentation issue)

How to test it?

Create a tenant,
Login with the tenant in incognito mode
remove the tenant.

Expected behavior: When page refreshed, user must logged out and user must be loggedout.

@mahmut-gundogdu mahmut-gundogdu added this to the 7.3-patch milestone Jul 24, 2023
@masum-ulu masum-ulu merged commit cfbdc6c into rel-7.3 Jul 24, 2023
1 of 2 checks passed
@masum-ulu masum-ulu deleted the issue/14735 branch July 24, 2023 12:56
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