[BUG] Investigate and clean up tenancy logic

[derek-ho]

What is the bug?
Currently, there are a few problems with the tenancy logic that lead to confusing behavior:

• Tenany popup related settings are stored in session/local storage. This may lead to unexpected and inconsistent behavior. Instead, this setting should just be computed on login, if it is not too heavy of an operation.
• The tenany popup is selected as global, even if you don't have access to that tenant, with the circle grayed out, and if you have unselected it you cannot reselect it
• The tenany icon is showing as "global" even if tenany is disabled.

All of these are side effects of complicated logic, so this issue is to clean up everything and make it easier to read and develop on.

How can one reproduce the bug?
Steps to reproduce the behavior:

1. Go to '...'
2. Click on '....'
3. Scroll down to '....'
4. See error

What is the expected behavior?
A clear and concise description of what you expected to happen.

What is your host/environment?

• OS: [e.g. iOS]
• Version [e.g. 22]
• Plugins

Do you have any screenshots?
If applicable, add screenshots to help explain your problem.

Do you have any additional context?
Add any other context about the problem.

[stephen-crawford]

[Triage] Hi @derek-ho, thanks for filing this issue. It sounds like you are mostly interested in refactoring the tenancy logic. Given the description, that seems like a good idea and we can go ahead and mark this issue as complete when the code has been refactored to a more concise state.