Skip to content

Implement Ticket Created and New Comment notifications #89

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

Merged
merged 15 commits into from
Jun 11, 2025

Conversation

ChrisPenner
Copy link
Member

@ChrisPenner ChrisPenner commented Jun 2, 2025

Overview

Implements the notification triggers/subscriptions for creating tickets, and new comments on tickets & contributions.

Also sketches out the types for Ticket/Contribution status updates, but those aren't done yet; that'll be next

Implementation notes

  • Adds database event topics for ticket created and ticket-comments and contribution-comments
  • Adds those topics to the watch_project topic group
  • Update the Share slack/discord feeds to track watch_project so they'll get these new updates.
  • Implements the chat-app messages for the new topics
  • Implements the hydrated data for the new topics (for hub and webhooks)
  • Re-organizes data payloads to make adding more events a little easier

Test coverage

  • Add the new notifications to existing notification transcripts

@ChrisPenner ChrisPenner changed the base branch from main to cp/topic-groups June 2, 2025 20:59
Base automatically changed from cp/topic-groups to main June 2, 2025 21:05
@ChrisPenner ChrisPenner force-pushed the cp/more-notification-topics branch from dcfcaaf to 2653797 Compare June 2, 2025 21:05
@ChrisPenner ChrisPenner force-pushed the cp/more-notification-topics branch from 7c235f5 to 9902fd1 Compare June 3, 2025 18:44
@ChrisPenner ChrisPenner merged commit f8c8916 into main Jun 11, 2025
8 checks passed
@ChrisPenner ChrisPenner deleted the cp/more-notification-topics branch June 11, 2025 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant