[Is a member of][User groups] Sync with API data #264
Conversation
carma12
force-pushed
the
au-member-of-cl
branch
2 times, most recently
from
6cdba56 to
a01c4ea
Compare
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
force-pushed
the
au-member-of-cl
branch
from
a01c4ea to
b944a7d
Compare
| error: userGroupsResult.error as unknown as FetchBaseQueryError, | ||
| }; | ||
| }, | ||
| }), | ||
| }), |
There was a problem hiding this comment.
In an unmerged PR I created a generic function for this. Otherwise we will have 10+ functions that all use the same code except for the "method". If it's ok, I suggest to wait for the "services" PR to go through, then we can refactor this PR (and others?) to use wrappers to invoke the generic "getting" function.
There was a problem hiding this comment.
In an unmerged PR I created a generic function for this. Otherwise we will have 10+ functions that all use the same code except for the "method". If it's ok, I suggest to wait for the "services" PR to go through, then we can refactor this PR (and others?) to use wrappers to invoke the generic "getting" function.
@carma12 Actually looking at your other PR's they all use this function, so that refactoring I mentioned above can be done afterwards in a new PR
|
Closing this PR in favor of this one: #289 |
The 'Is a member of' > 'User groups' table has been adapted to replace the dummy data with the real one from the IPA server.