Skip to content

Conversation

@MelvinMartins-NIST
Copy link
Contributor

  • Implementation of the web layer for group related functions
  • Implementation of the unit test for the endpoints related to groups

Endpoints include:
POST: /midas/group/grp0 to create a new group (group id should be sent back by dbio)
POST: /midas/group/grp0/[grp_id] to add users to an existing group
PUT: /midas/group/grp0/[grp_id] to change the whole list of users
DELETE: /midas/group/grp0/[grp_id] to delete group
DELETE: /midas/group/grp0/[grp_id]/[user] to delete user from a group
GET: /midas/group/grp0/[grp_id] to get info from a group

RayPlante and others added 30 commits August 15, 2024 09:20
   * make _add_applied easier to use by returning the created issue
   * define __all__ for easier conversion of nistoar.pdr.preserve.bagit.validate
   * fix handling of **kw args
   * fix handling of execution errors in ValidationBase
DBIO: Enable bulk permission updates
Copy link
Collaborator

@RayPlante RayPlante left a comment

Choose a reason for hiding this comment

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

Still testing, but I wanted to say that this is really looking great: I quickly understood whats going on just from reading the code. Thanks!

@MelvinMartins-NIST
Copy link
Contributor Author

@RayPlante Fixes completed, all endpoints tested and working as expected on my end.
As a reminder, Endpoints include:
POST: /midas/group/grp0 to create a new group (group id should be sent back by dbio)
POST: /midas/group/grp0/[grp_id] to add users to an existing group
PUT: /midas/group/grp0/[grp_id] to change the whole list of users
DELETE: /midas/group/grp0/[grp_id] to delete group
DELETE: /midas/group/grp0/[grp_id]/[user] to delete user from a group
GET: /midas/group/grp0/[grp_id] to get info from a group

Branch ready to be reviewed

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.

5 participants