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

fix(rest): modified accept request to see the proposed changes in user to moderation request rest Api. #2517

Conversation

nikkuma7
Copy link

@nikkuma7 nikkuma7 commented Jul 16, 2024

Closes: #2490, #2518
###Description

Upon clicking the "Accept Request" button for a moderation request, the REST API response returns a 202 (Accepted) status. However, the database is not being updated with the proposed changes. For example, if a project page is edited and a moderation request is created, accepting the proposed changes does not update the project page with the new values; it still shows the old values.

Please provide a summary of your changes here.

  • Which issue is this pull request belonging to and how is it solving it? (Refer to issue here)
  • Did you add or update any new dependencies that are required for your change?

Issue:

Suggest Reviewer

You can suggest reviewers here with an @mention.

How To Test?

How should these changes be tested by the reviewer?
Have you implemented any additional tests?

Checklist

Must:

  • All related issues are referenced in commit messages and in PR

@GMishx GMishx added needs code review needs general test This is general testing, meaning that there is no org specific issue to check for New-UI Level for the API and UI level changes for the new-ui labels Jul 16, 2024
Copy link
Contributor

@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.

Minor changes needed.

@nikkuma7 nikkuma7 force-pushed the fix/db-update-on-moderation-accept-for-user branch from e3d48e1 to 6ea1408 Compare August 1, 2024 06:26
Copy link
Contributor

@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.

@GMishx
Copy link
Contributor

GMishx commented Aug 31, 2024

@amritkv can you please test this PR?

@amritkv
Copy link

amritkv commented Sep 2, 2024

@amritkv can you please test this PR?

Hi @nikkuma7 & @GMishx !

Build is failing for this PR because of liferay inclusion.

image

… request.

Signed-off-by: Nikesh kumar <kumar.nikesh@simens.com>
@GMishx GMishx force-pushed the fix/db-update-on-moderation-accept-for-user branch from 6ea1408 to 5943127 Compare September 2, 2024 13:21
@GMishx
Copy link
Contributor

GMishx commented Sep 2, 2024

The branch is now rebased with latest main @amritkv , please check again.

@amritkv
Copy link

amritkv commented Sep 3, 2024

The branch is now rebased with latest main @amritkv , please check again.

Hi @GMishx !
Yes, testing is successful. The fields are getting updated in project table. 👍🏻

@GMishx GMishx added ready ready to merge and removed needs general test This is general testing, meaning that there is no org specific issue to check for labels Sep 3, 2024
@GMishx GMishx merged commit 159ad6a into eclipse-sw360:main Sep 3, 2024
1 of 2 checks passed
@GMishx GMishx deleted the fix/db-update-on-moderation-accept-for-user branch September 3, 2024 05:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
New-UI Level for the API and UI level changes for the new-ui ready ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MR is not updating database over rest api call
3 participants