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

Information bubble #3887

Open
wants to merge 8 commits into
base: ng
Choose a base branch
from
Open

Information bubble #3887

wants to merge 8 commits into from

Conversation

hipek8
Copy link
Contributor

@hipek8 hipek8 commented Mar 18, 2025

Limit visibility of hosts (data center asset, virtual server, cluster, cloud host) for users in information bubble
User in an information bubble or in a group which is in an information bubble will see limited assets - asset must belong to the service env of a service which belongs to at least 1 of user's information bubble.
BaseObjectsSupport visibility is also limited (but not Support visibility as it's less clear how to do that)

migrations.RemoveField(
model_name='servicebasedinformationbubble',
name='groups',
),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so migration 0002 is not really needed?

@vi4m
Copy link
Contributor

vi4m commented Mar 18, 2025

I know this PR is almost finished. I'm afraid, Information Bubble is non standard and quite confusing name, isn't it ? Did we evaluate other choices ?

A few options:

  • visibilityScope (it's precise and fits the purpose I think)

Alternatives

  • backOfficeRegion
  • accessZone

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.

4 participants