getsentry / getsentry/sentry

App Modules - Consolidate notification logic

Open
#73,858 4 comments 0 reactions 0 assignees View on GitHub
Backend (Infra & Debt) Task
Dominant language
Python
Stars
44.8k
Forks
4.9k
Avg merge
22h 21m
Merged PRs (30d)
586

Description

Notifications are primarily control_silo scoped resources. Currently the components that comprise the control silo aspects of notifications are spread throughout the repository. Consolidating those classes and functions into sentry.notifications will help clarify the application's design and make running subsets of tests easier and faster.

- Move endpoints, serializers, models into sentry.notifcations

We'll need to provide import shims for any symbols used in getsentry until those imports can be updated separately.

cc @Christinarlong

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.