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 attachment info in response to the moderation request rest api #2498

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

keerthi-bl
Copy link
Contributor

Closes : #2497

Description

Please describe your issue in few words here.

How to reproduce

Describe the bug and list the steps you used when the issue occurred.

Screenshots ( if applicable )

image

@keerthi-bl keerthi-bl force-pushed the fix/getAttachmentsInModReqEndPoint branch 2 times, most recently from 9a986d0 to 9c63726 Compare July 10, 2024 09:53
@rudra-superrr
Copy link
Contributor

Resolve the merge conflicts

@keerthi-bl keerthi-bl force-pushed the fix/getAttachmentsInModReqEndPoint branch from c8fb304 to bcddf76 Compare October 21, 2024 05:32
@GMishx GMishx added needs code review needs general test This is general testing, meaning that there is no org specific issue to check for and removed has merge conflicts labels Oct 21, 2024
@GMishx
Copy link
Member

GMishx commented Oct 21, 2024

@rudra-superrr please test this branch.

@smrutis1 please help review it.

@rudra-superrr
Copy link
Contributor

Testing was successful.
image

@rudra-superrr
Copy link
Contributor

One thing that I have found is, when a user updates a project then MR is created but if the same user updates some other field of the same project then it overwrites the previous moderation request.

@GMishx GMishx removed the needs general test This is general testing, meaning that there is no org specific issue to check for label Oct 24, 2024
@GMishx
Copy link
Member

GMishx commented Oct 24, 2024

One thing that I have found is, when a user updates a project then MR is created but if the same user updates some other field of the same project then it overwrites the previous moderation request.

Can you please open an issue for the same? We need to rework on the Moderation Request with REST API. With the Liferay UI, we had the liberty to check when an MR will be created and how to treat it. But with REST API, currently there is no way to check the same and we have to revert to this behavior.

Please check #2405 (comment) as well.

@rudra-superrr
Copy link
Contributor

One thing that I have found is, when a user updates a project then MR is created but if the same user updates some other field of the same project then it overwrites the previous moderation request.

Can you please open an issue for the same? We need to rework on the Moderation Request with REST API. With the Liferay UI, we had the liberty to check when an MR will be created and how to treat it. But with REST API, currently there is no way to check the same and we have to revert to this behavior.

Please check #2405 (comment) as well.

But this issue is happening because of the changes made in this PR.
Please have a look into this @keerthi-bl

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Need modified attachment info in response to the moderation request rest API
3 participants