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 Ticket message file #31120

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

MaximilienR-easya
Copy link
Contributor

Fix

Fix the file linked to ticket message on the document page.

Now we have a separate list for the files linked to the messages for more visibility, and we can modify or delete them without problem.

@eldy
Copy link
Member

eldy commented Sep 26, 2024

How do you reproduce the bug ?

@eldy eldy added the Bug or PR need more information This bug or PR needs more information (answer to a question or more accurate description) label Sep 26, 2024
@MaximilienR-easya
Copy link
Contributor Author

How do you reproduce the bug ?

It's in accord with this PR #30554 :

Futhermore, edit and delete files doesn't work due to the wrong module part passed into list of documents method.
The module part is "actions" for the attached files in a message sent by mail.
The module part "ticket" is correct when you join a file from the ticket card.
...
In the case of an attached file in an event, the module part is "actions" (not "ticket") and I prevent the user from
edit / delete theses files directly from the ticket card. He can do it from the linked event.

I fixed it to be able to do modify /delete without problem the files linked to the messages

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug or PR need more information This bug or PR needs more information (answer to a question or more accurate description)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants