Skip to content

feat(np): Impl Discord send#99835

Merged
Christinarlong merged 20 commits into
masterfrom
crl/discord-send
Sep 30, 2025
Merged

feat(np): Impl Discord send#99835
Christinarlong merged 20 commits into
masterfrom
crl/discord-send

Conversation

@Christinarlong

@Christinarlong Christinarlong commented Sep 18, 2025

Copy link
Copy Markdown
Contributor

Impl. send method for discord provider
relies on: #99713

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Sep 18, 2025
@codecov

codecov Bot commented Sep 18, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 78.57143% with 3 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
.../sentry/notifications/platform/discord/provider.py 57.14% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master   #99835      +/-   ##
==========================================
+ Coverage   76.46%   81.06%   +4.60%     
==========================================
  Files        8599     8641      +42     
  Lines      380952   383410    +2458     
  Branches    23932    24100     +168     
==========================================
+ Hits       291299   310830   +19531     
+ Misses      89291    72227   -17064     
+ Partials      362      353       -9     

Comment thread src/sentry/notifications/platform/discord/provider.py
Base automatically changed from crl/add-slack-send to master September 30, 2025 17:23
@Christinarlong Christinarlong merged commit 5a9dbc6 into master Sep 30, 2025
65 checks passed
@Christinarlong Christinarlong deleted the crl/discord-send branch September 30, 2025 22:11
jerryzhou196 pushed a commit that referenced this pull request Oct 1, 2025
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 16, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants