-
Notifications
You must be signed in to change notification settings - Fork 400
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(API): list groups that can be deleted #2247
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some suggestions
@dushimsam , can you please fix the branch so we can merge it? |
4262e75
to
6a8601a
Compare
Done, you can review and let me know what's next to update if still available. |
24443ab
to
b1292aa
Compare
Completed tests for this controller. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes looks good.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes looks good. Working as expected.
Wow, thanks a lot sir. |
b1292aa
to
d128a6b
Compare
d128a6b
to
e0d5275
Compare
tested and working perfectly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A tiny change. Missed it earlier.
e0d5275
to
fa56840
Compare
This pull request has conflicts, please rebase with master to resolve those before we can evaluate the pull request. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dushimsam Kindly rebase the branch with upstream master to resolve the merge conflicts.
fa56840
to
ca4c68a
Compare
ca4c68a
to
997e549
Compare
Ready now sir, kindly review and merge for me. |
997e549
to
99aa362
Compare
Signed-off-by: dushimsam dushsam100@gmail.com
Description
To check if the user has any group that is possible to be deleted.
Changes
GroupController
. to check if the group exists.index.php
) by adding a new route/groups/has-deletable-group
.openapi.yaml
file to introduce a new API.How to test
http://localhost/repo/api/v1/groups/has-deletable-group
The following property in the response body called
available
should be marked as true if you have any group that is deletable , else it will always be marked as false.By default you have group
fossy
, this one is not deletable , the deletable groups are the one you add afterwards.In other words , by default you don't have any deletable group , unless you start adding your own groups.
Related Issue:
Fixes #2246
cc: @shaheemazmalmmd @GMishx