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

[Issue Tracker] Issue Change Notifications #8885

Merged
merged 5 commits into from
Jan 11, 2024
Merged

[Issue Tracker] Issue Change Notifications #8885

merged 5 commits into from
Jan 11, 2024

Conversation

skarya22
Copy link
Contributor

Brief summary of changes

  • Fixed issue where the assignee dropdown did not show the current assignee
  • Emails users who were added as watching
  • When a comment is added, it adds it to the email to everyone watching

Testing instructions (if applicable)

  1. Open the issue tracker and add a user with your email address as the watcher to an issue
  2. Confirm that you received an email about the issue
  3. Try changing various fields and confirm an email is sent for each with some context.

Link(s) to related issue(s)

CCNA Overrides:
https://github.com/aces/CCNA/pull/3889
https://github.com/aces/CCNA/pull/4618

Old PR To Loris Core Main:
#8850

@skarya22 skarya22 mentioned this pull request Sep 18, 2023
Closed
kongtiaowang
kongtiaowang approved these changes Dec 12, 2023
View reviewed changes
Copy link
Contributor

@kongtiaowang kongtiaowang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kongtiaowang kongtiaowang added the Passed manual tests PR has been successfully tested by at least one peer label Dec 12, 2023
@skarya22 skarya22 linked an issue Dec 12, 2023 that may be closed by this pull request
[issue_tracker] The Assignee is not populated when updating and issue (also preventing the page to save) #8872
Closed
@driusan driusan merged commit 03d2092 into aces:24.1-release Jan 11, 2024
9 checks passed
@ridz1208 ridz1208 added this to the 24.1.6 milestone Apr 9, 2024
@christinerogers christinerogers mentioned this pull request May 6, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kongtiaowang kongtiaowang kongtiaowang approved these changes

Assignees

@kongtiaowang kongtiaowang

Labels
Passed manual tests PR has been successfully tested by at least one peer
Projects
None yet
Milestone
24.1.6
Development

Successfully merging this pull request may close these issues.

[issue_tracker] The Assignee is not populated when updating and issue (also preventing the page to save)
5 participants
@skarya22 @kongtiaowang @driusan @ridz1208 @CamilleBeau