Skip to content

Conversation

malwilley
Copy link
Member

  • Fixes casing for query params on open periods endpoint (detectorId vs detector_id)
  • Adds doc decorators

@malwilley malwilley requested a review from a team as a code owner August 26, 2025 23:01
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Aug 26, 2025
Copy link

codecov bot commented Aug 26, 2025

Codecov Report

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

Files with missing lines Patch % Lines
...flow_engine/endpoints/organization_open_periods.py 64.28% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master   #98315      +/-   ##
==========================================
+ Coverage   80.83%   80.99%   +0.15%     
==========================================
  Files        8564     8564              
  Lines      376740   376746       +6     
  Branches    24144    24144              
==========================================
+ Hits       304555   305142     +587     
+ Misses      71800    71219     -581     
  Partials      385      385              

@malwilley malwilley merged commit a710b6f into master Aug 27, 2025
64 checks passed
@malwilley malwilley deleted the malwilley/fix/aci-open-periods-casing branch August 27, 2025 16:30
chromy pushed a commit that referenced this pull request Aug 27, 2025
- Fixes casing for query params on open periods endpoint (`detectorId`
vs `detector_id`)
- Adds doc decorators
kenzoengineer pushed a commit that referenced this pull request Aug 28, 2025
- Fixes casing for query params on open periods endpoint (`detectorId`
vs `detector_id`)
- Adds doc decorators
andrewshie-sentry pushed a commit that referenced this pull request Aug 28, 2025
- Fixes casing for query params on open periods endpoint (`detectorId`
vs `detector_id`)
- Adds doc decorators
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