Skip to content

Comments

v1.8.5#124

Merged
gmpassos merged 1 commit intomasterfrom
UNAUTHORIZED-error-response
Dec 25, 2024
Merged

v1.8.5#124
gmpassos merged 1 commit intomasterfrom
UNAUTHORIZED-error-response

Conversation

@gmpassos
Copy link
Contributor

  • refactor(api):

    • 🔨 improve payload handling in resolveBody and resolveBodySync
    • 🔨 UNAUTHORIZED, BAD_REQUEST: Allow error as payload in response generation.
  • swiss_knife: ^3.2.3

  • yaml: ^3.1.3

  • stream_channel: ^2.1.3

- refactor(api):
  - 🔨 improve payload handling in resolveBody and resolveBodySync
  - 🔨 `UNAUTHORIZED`, `BAD_REQUEST`: Allow `error` as `payload` in response generation.

- swiss_knife: ^3.2.3
- yaml: ^3.1.3
- stream_channel: ^2.1.3
@gmpassos gmpassos merged commit f59e224 into master Dec 25, 2024
3 checks passed
@codecov
Copy link

codecov bot commented Dec 25, 2024

Codecov Report

Attention: Patch coverage is 28.57143% with 5 lines in your changes missing coverage. Please review.

Project coverage is 65.44%. Comparing base (67c392c) to head (61b65d9).
Report is 2 commits behind head on master.

Files with missing lines Patch % Lines
lib/src/bones_api_server.dart 28.57% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #124      +/-   ##
==========================================
- Coverage   65.45%   65.44%   -0.01%     
==========================================
  Files          60       60              
  Lines       19828    19830       +2     
==========================================
  Hits        12978    12978              
- Misses       6850     6852       +2     
Flag Coverage Δ
unittests 65.44% <28.57%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@gmpassos gmpassos deleted the UNAUTHORIZED-error-response branch March 10, 2025 00:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant