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

feat(api): Update customise data #2545

Merged
merged 3 commits into from
Sep 28, 2023
Merged

Conversation

soham4abc
Copy link
Contributor

@soham4abc soham4abc commented Jul 17, 2023

Description

Added update feature for /customise endpoint

Screenshots

image

How to test

Send a PUT request to /customise endpoint with the following body:

{
  "key": "key_name",
  "value":"updated value"
}

Closes #2519

NOTE : MERGE ONLY AFTER MERGING #2520

@GMishx @shaheemazmalmmd @avinal

@soham4abc soham4abc changed the title Api/set customise feat(api): Update customise data Jul 18, 2023
@github-actions github-actions bot added the has merge conflicts PR to be rebased label Jul 24, 2023
@github-actions
Copy link

This pull request has conflicts, please rebase with master to resolve those before we can evaluate the pull request.

@github-actions github-actions bot added has merge conflicts PR to be rebased and removed has merge conflicts PR to be rebased labels Jul 25, 2023
@github-actions
Copy link

This pull request has conflicts, please rebase with master to resolve those before we can evaluate the pull request.

@GMishx GMishx added the GSOC-23 Label to tag pull request which are part of the GSOC 2023 label Jul 28, 2023
@soham4abc soham4abc force-pushed the api/setCustomise branch 2 times, most recently from 2b97cbd to c6fdf67 Compare July 28, 2023 18:01
@github-actions github-actions bot removed the has merge conflicts PR to be rebased label Jul 28, 2023
@github-actions github-actions bot added the has merge conflicts PR to be rebased label Aug 8, 2023
@github-actions
Copy link

github-actions bot commented Aug 8, 2023

This pull request has conflicts, please rebase with master to resolve those before we can evaluate the pull request.

src/lib/php/Dao/SysConfigDao.php Outdated Show resolved Hide resolved
src/www/ui/api/Controllers/CustomiseController.php Outdated Show resolved Hide resolved
src/www/ui/api/Controllers/CustomiseController.php Outdated Show resolved Hide resolved
src/www/ui/api/documentation/openapi.yaml Outdated Show resolved Hide resolved
src/www/ui/api/documentation/openapi.yaml Outdated Show resolved Hide resolved
src/www/ui/api/documentation/openapi.yaml Outdated Show resolved Hide resolved
src/www/ui/api/documentation/openapi.yaml Outdated Show resolved Hide resolved
@github-actions github-actions bot added the has merge conflicts PR to be rebased label Sep 27, 2023
@github-actions
Copy link

This pull request has conflicts, please rebase with master to resolve those before we can evaluate the pull request.

Signed-off-by: Gaurav Mishra <mishra.gaurav@siemens.com>
Copy link
Member

@GMishx GMishx left a 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.

@GMishx GMishx added the ready label Sep 28, 2023
@GMishx GMishx merged commit 7136cfa into fossology:master Sep 28, 2023
13 checks passed
@GMishx GMishx deleted the api/setCustomise branch September 28, 2023 10:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GSOC-23 Label to tag pull request which are part of the GSOC 2023 ready
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Customize
3 participants