-
Notifications
You must be signed in to change notification settings - Fork 0
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
New notifications #16
base: develop
Are you sure you want to change the base?
Conversation
…pdesk for Colorado
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the fixes, it's working great! I have some other notes:
- Add notification messages for attaching a file; changing the priority; changing the due date; changing the title
- The message "A ticket has been assigned to you" should be changed out for the message "You've been assigned to ticket <id> by <name>"
One more task: create a management command that when run, deletes read notifications that are over a day old, and any expired announcements. Management commands are run with This command won't need any options, and we have a system that can run commands every x minutes on our servers, so don't worry about when it should run. The command should use the current time (like |
…ses when ticket is changed
Any background context you want to provide?
What's this PR do?
When should it be merged in?
Do you want someone to review this before it gets merged in?
How should this be manually tested?
What are the relevant tickets?
Screenshots (if appropriate)