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

[Is a member of][User groups] 'Add' button functionality #272

Closed
wants to merge 4 commits into from

Conversation

carma12
Copy link
Collaborator

@carma12 carma12 commented Feb 7, 2024

The 'Add' button functionality allows to associate any user group's member to a given user.

To ensure that an Alert component will be displayed after a new addition, a new interface AlertObject has been created to refer to a given alerts (defined as const alerts = useAlerts()) that is passed through props to a children component.

This PR depends on this one to be merged: #264

@carma12 carma12 requested review from pvoborni and mreynolds389 and removed request for pvoborni February 7, 2024 09:36
The `User` data type needs the following
parameters that will be managed in the
'Is a member of' section:
- `memberof_group`
- `memberof_netgroup`
- `memberof_role`
- `memberof_hbacrule`
- `memberof_sudorule`
- `memberof_subid`

Signed-off-by: Carla Martinez <[email protected]>
As other components are using the `MemberOfTable` and
`MemberOfToolbar` components, there are many dependencies
to correct and adapt. The new `MemberOfTableNew` and
`MemberOfToolbarNew` have been defined to be used
when adapting the 'User groups' data only.

The same has been done in the data types by
creating a new `UserGroupNew` type that will be
used later.

The `normalizeString` utility function is meant to
normalize LDAP values from `string[]` to `string`.

Signed-off-by: Carla Martinez <[email protected]>
The dummy data must be replaced by the
data from the API for the 'User groups'
only.

Signed-off-by: Carla Martinez <[email protected]>
@carma12 carma12 force-pushed the member-of-user-groups-add branch from 2264e75 to 9618dde Compare February 7, 2024 09:38
The 'Add' button functionality allows
to associate any user group's member
to a given user.

Signed-off-by: Carla Martinez <[email protected]>
@carma12
Copy link
Collaborator Author

carma12 commented Feb 28, 2024

Closing this PR in favor of this one: #298

@carma12 carma12 closed this Feb 28, 2024
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.

1 participant