Skip to content
This repository has been archived by the owner on Nov 14, 2024. It is now read-only.

refactor(api): on fait passer les paramètres dans un objet plutôt qu'à plat #1464

Merged
merged 1 commit into from
Sep 16, 2024

Conversation

MichaelBitard
Copy link
Contributor

@MichaelBitard MichaelBitard commented Sep 16, 2024

ils ont mis ça en place sur ma mission du vendredi, et je trouve ça plus lisible, et on peut ignorer les paramètres qu'on n'utilise pas

@MichaelBitard MichaelBitard added the 🤓 tech Difficile à recetter pour un non-tech label Sep 16, 2024
@codecov-commenter
Copy link

codecov-commenter commented Sep 16, 2024

Codecov Report

Attention: Patch coverage is 87.34177% with 10 lines in your changes missing coverage. Please review.

Project coverage is 80.83%. Comparing base (d139c9b) to head (18c22a5).

Files with missing lines Patch % Lines
packages/api/src/api/rest/administrations.ts 78.72% 10 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1464      +/-   ##
==========================================
- Coverage   80.83%   80.83%   -0.01%     
==========================================
  Files         729      729              
  Lines       69729    69724       -5     
  Branches     6576     6578       +2     
==========================================
- Hits        56366    56361       -5     
  Misses      13269    13269              
  Partials       94       94              
Flag Coverage Δ
api 78.23% <87.17%> (-0.01%) ⬇️
api-integration 72.46% <87.17%> (-0.01%) ⬇️
api-unit 33.95% <0.00%> (+<0.01%) ⬆️
common 96.56% <ø> (ø)
ui 73.23% <100.00%> (ø)

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.

@MichaelBitard MichaelBitard merged commit 9658519 into master Sep 16, 2024
21 checks passed
@MichaelBitard MichaelBitard deleted the refactor-new-apis branch September 16, 2024 12:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🤓 tech Difficile à recetter pour un non-tech
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants