Skip to content

Add ticket sync methods to Notification interface#90

Draft
noah-tutt-praetorian wants to merge 1 commit intomainfrom
nmt/freshservice
Draft

Add ticket sync methods to Notification interface#90
noah-tutt-praetorian wants to merge 1 commit intomainfrom
nmt/freshservice

Conversation

@noah-tutt-praetorian
Copy link
Collaborator

Summary

  • Extends Notification interface with ticket sync methods
  • Enables generic ticket sync cron for multiple ticketing integrations
  • Non-breaking change with default implementations in XYZ

Changes

  • Add UpdateTicketMetadata(attr Attribute) (Attribute, error) method
  • Add SupportsTicketSync() bool method
  • Supports auto-discovery of ticket integrations in cron job

Related

Part of Freshservice integration implementation. Chariot PR will follow.

Test Plan

  • Tabularium builds successfully
  • Chariot backend builds with new interface
  • Jira integration continues to work
  • Freshservice integration works with new interface

Extends Notification interface with UpdateTicketMetadata and SupportsTicketSync
methods to enable generic ticket sync cron job across multiple ticketing
integrations (Jira, Freshservice, etc).

Changes:
- Add UpdateTicketMetadata(attr Attribute) (Attribute, error)
- Add SupportsTicketSync() bool
- Base XYZ implementation provides "not supported" defaults
- Ticketing integrations override to enable sync functionality
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

Comments