-
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): delete admin-license candidate #2298
feat(API): delete admin-license candidate #2298
Conversation
This pull request has conflicts, please rebase with master to resolve those before we can evaluate the pull request. |
9c8381d
to
e2966d8
Compare
e0d2b9e
to
5dc7231
Compare
This pull request has conflicts, please rebase with master to resolve those before we can evaluate the pull request. |
5dc7231
to
d91acd0
Compare
d91acd0
to
22f9e48
Compare
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.
Tested, working as expected.
Signed-off-by: Gaurav Mishra <mishra.gaurav@siemens.com>
22f9e48
to
644879d
Compare
Signed-off-by: dushimsam dushsam@gmail.com
Description
An endpoint to delete admin-license candidate.
Changes
LicenseController
.AdminLicenseCandidate.php
to be reusable by the outside callers.index.php
) by adding a new routeDELETE
/license/candidates/{id}
.openapi.yaml
file to introduce a new API.How to test
Make a DELETE request on the endpoint accessed at "/license/candidates/{id}"
Example
1. FIRST CASE
In this case, the Admin-license-candidate exists, and the license-candidate will be deleted successfully.
2. SECOND CASE
In this case, the Admin-license-candidate doesn't exist, so 404 error will be returned instead.
3. THIRD CASE
The request is not coming from the Admin.
This request can only be run by the admin , else the request will fail returning 400 ERROR.
Related Issue:
Fixes #2297
cc: @shaheemazmalmmd @GMishx