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

feat: angular migration- groups #10853

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aditya-goyal01
Copy link
Contributor

@aditya-goyal01 aditya-goyal01 commented Feb 27, 2025

Issue

https://gravitee.atlassian.net/browse/APIM-2492
https://gravitee.atlassian.net/browse/APIM-6011

Description

UI changes

(https://gravitee.atlassian.net/browse/APIM-2492)

  1. Standalone components for groups and related dialogs. Components are directly mapped to routes.
  2. Migration of old groups and related components to Angular.
  3. Cleaner and consistent UI with separation of tabs for Members, Invitations.
  4. Moved actions in separate cards along with the data they belong to.
  5. Combined few general information and settings in groups into single card with required sections instead of multiple cards.
  6. Added functions to search users while adding, editing and deleting members.
  7. Separated cards for APIs and Applications data and related operations.
  8. Added paginated data tables for all list data (only in front-end) with search function.
  9. Removed unnecessary old files.

(https://gravitee.atlassian.net/browse/APIM-6011)

  1. Created a UI element for the setting required to enable enforce adding at least one group to the applications.
  2. Added a drop-down list of groups to select and add to the application in the creation screen. When the setting is enabled, the selection of at least one group is mandatory to create the application.

Backend changes

(https://gravitee.atlassian.net/browse/APIM-6011)

  1. Added a key in console settings and configuration to hold the boolean value of setting.
  2. Throws bad request exception when trying to edit an application that have no groups attached prior to enabling the setting.

Screens

Groups list

Screenshot 2025-03-05 at 7 05 24 PM

Create new group

Screenshot 2025-03-05 at 7 07 16 PM

Edit group

  1. General Settings
Screenshot 2025-03-05 at 7 08 11 PM
  1. Members and invitations

image

  1. APIs and Applications
Screenshot 2025-03-05 at 7 09 22 PM

Delete Group Dialog

image

Add members

  1. Options

image

  1. Dialogs

image

image

Edit member

image

  1. Upgrade role

image

  1. Downgrade role

image

image

Invitations

  1. List invitations

image

  1. Delete invitation

image

  1. Invitation dialog

image

Enable at least one group required in application

image

Groups while creating application

image

Error when setting is enabled and editing an application

image

Additional context

The back-end resources & URL migrations of groups and related operations are not included in this pull request.


📚  View the storybook of this branch here

@aditya-goyal01 aditya-goyal01 force-pushed the apim-2492-angular-migration-groups branch from 608b0ad to 10ca272 Compare February 27, 2025 17:45
@aditya-goyal01 aditya-goyal01 changed the title (feat) angular migration- groups feat: angular migration- groups Feb 27, 2025
@aditya-goyal01 aditya-goyal01 force-pushed the apim-2492-angular-migration-groups branch 28 times, most recently from 6a68bda to 72784bb Compare March 4, 2025 11:39
@aditya-goyal01 aditya-goyal01 force-pushed the apim-2492-angular-migration-groups branch 28 times, most recently from 802f6fa to 2b0e009 Compare March 12, 2025 15:21
Signed-off-by: aditya.goyal <aditya.goyal@graviteesource.com>

feat: admin can require user groups to be added in applications

Signed-off-by: aditya.goyal <aditya.goyal@graviteesource.com>

feat: resolved review comments

Signed-off-by: aditya.goyal <aditya.goyal@graviteesource.com>
@aditya-goyal01 aditya-goyal01 force-pushed the apim-2492-angular-migration-groups branch from 2b0e009 to c327573 Compare March 12, 2025 15:36
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