Skip to content

[Maintenance] - Schema Changes Move Notifications to Event Level #4090

@cielbellerose

Description

@cielbellerose

Description

Setting if Slack notifications are sent for calendar events currently lives on Event_Type and controls whether Slack notifications are sent for all events of that type. We want to move sendSlackNotifications down to the Event level so users can toggle notifications per event when creating/editing calendar events.

sendSlackNotifications should be fully removed from Event_Type, and added to Event

Proposed Solution

  • Schema Changes: Remove sendSlackNotifications from Event_Type and add to the Event model + migration
  • Shared Type Move sendSlackNotifications from EventType to Event
  • Calendar Routes Remove sendSlackNotifications from event-type POST create and edit, add to event
  • Calendar Controller Remove from createEventType/editEventType and add to createEvent/editEvent
  • ** Calendar Service** Remove from event type functions, add to event
    • markUserConfirmed needs to use Event not Event Type to send slack user confirmation
  • ** Calendar Transformer**
  • ** Notification Service** daily Slack notification job needs to filter using Event instead of Event Type for sendSlackNotifications
  • Update tests

Acceptance Criteria

  • sendSlackNotifications moved from Event_Type to Event for route, controller, service, transformer, and in prisma & shared type
  • schema migration
  • Postman tests for updated event create and event edit routes
  • tests passing

Metadata

Metadata

Assignees

Type

Projects

Status

Under Review

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions