Skip to content

Conversation

@edwardgou-sentry
Copy link
Contributor

Adds preferredPolarity attribute to the widget ThresholdType serializer to enable choosing direction of big number threshold config

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Dec 23, 2025
@codecov
Copy link

codecov bot commented Dec 23, 2025

Codecov Report

❌ Patch coverage is 92.30769% with 1 line in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/sentry/api/serializers/models/dashboard.py 90.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           master   #105407      +/-   ##
===========================================
- Coverage   80.58%    80.57%   -0.01%     
===========================================
  Files        9441      9441              
  Lines      404940    404953      +13     
  Branches    25684     25684              
===========================================
+ Hits       326304    326306       +2     
- Misses      78167     78178      +11     
  Partials      469       469              



class ThresholdType(TypedDict):
class ThresholdTypeOptional(TypedDict, total=False):
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Creating a class specifically for the optional preferredPolarity because thats what the openapi generator recognizes

@edwardgou-sentry edwardgou-sentry marked this pull request as ready for review December 23, 2025 03:20
@edwardgou-sentry edwardgou-sentry requested a review from a team as a code owner December 23, 2025 03:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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