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.

Sign up for GitHub

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

Jump to bottom

FIX user notification for ticket assigned on creation #25866

Closed
wants to merge 21 commits into from
Closed

FIX user notification for ticket assigned on creation #25866

wants to merge 21 commits into from

Conversation

BenjaminFlr
Copy link
Contributor

When an user is assigned to a ticket directly during the creation, there is no email notification whereas there is one when the user is assigned AFTER the creation.

I propose this little change in order to fix it and notify an user even when he's assigned during the ticket creation.

@eldy eldy added the PR to fix - See feedback in comments PR needs to be fixed to be integrated (some comments should describes the fix to do) label Sep 11, 2023
@BenjaminFlr
Copy link
Contributor Author

I made the changes.
@eldy : it should now do the trick.

@BenjaminFlr
Copy link
Contributor Author

Any update on here?

@BenjaminFlr BenjaminFlr requested a review from eldy April 11, 2024 14:15
@BenjaminFlr
Copy link
Contributor Author

@eldy all the pre-commit tests are now ok :)

@BenjaminFlr
Copy link
Contributor Author

Little up here @eldy :)

@W1W1-M
Copy link
Contributor

W1W1-M commented Sep 18, 2024 •
edited
Loading

Hello,
This fix would be a great "little" addition for ticketing system !

@IC-William IC-William mentioned this pull request Nov 15, 2024
Merged
@eldy
Copy link
Member

eldy commented Nov 19, 2024

Replaced by #31928 that has no conflict.

@eldy eldy closed this Nov 19, 2024
@BenjaminFlr BenjaminFlr deleted the assignNotif branch November 23, 2024 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eldy eldy eldy left review comments

Assignees
No one assigned
Labels
PR to fix - See feedback in comments PR needs to be fixed to be integrated (some comments should describes the fix to do)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@BenjaminFlr @W1W1-M @eldy