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

Add Publisher and Admin UI changes for GW visibility feature #848

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

npamudika
Copy link
Contributor

@npamudika npamudika commented Jan 10, 2025

Fixes related to wso2/api-manager#3378

This PR adds a new column with the GW visibility permissions to the Deployments page in Publisher as below.

PubUI-1 PubUI-2 PubUI-3

The Add Gateway Environment page in Admin portal will be modified with a new Permission field as below.

AdminUI-4 AdminUI-5

The roles can also be added as below.

AdminUI-6

The Gateway listing page in Admin page will be modified with a new Permission column as below.

AdminUI-1 AdminUI-2 AdminUI-3

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Naduni Pamudika seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@npamudika npamudika changed the title Add Publisher UI changes for GW visibility feature Add Publisher and Admin UI changes for GW visibility feature Jan 22, 2025
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
20.9% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

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