Skip to content

Correctif : utilise le bon type pour le nombre de dossiers par démarche dans le manager #11720

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 5, 2025

Conversation

E-L-T
Copy link
Contributor

@E-L-T E-L-T commented Jun 4, 2025

Copy link

codecov bot commented Jun 4, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 58.68%. Comparing base (ab84fbf) to head (d6d691b).
Report is 2 commits behind head on main.

❗ There is a different number of reports uploaded between BASE (ab84fbf) and HEAD (d6d691b). Click for more details.

HEAD has 8 uploads less than BASE
Flag BASE (ab84fbf) HEAD (d6d691b)
16 8
Additional details and impacted files
@@             Coverage Diff             @@
##             main   #11720       +/-   ##
===========================================
- Coverage   74.61%   58.68%   -15.93%     
===========================================
  Files        1267     1267               
  Lines       27829    27829               
  Branches     5734     5734               
===========================================
- Hits        20764    16332     -4432     
- Misses       7065    11497     +4432     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@E-L-T E-L-T marked this pull request as ready for review June 4, 2025 08:20
Copy link
Member

@colinux colinux left a comment

Choose a reason for hiding this comment

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

pourquoi pas mettre Field::Number ?

@E-L-T E-L-T force-pushed the fix-dossiers-count-bug-in-manager branch from 9f382f7 to c87cdb8 Compare June 5, 2025 07:26
@E-L-T
Copy link
Contributor Author

E-L-T commented Jun 5, 2025

pourquoi pas mettre Field::Number ?

en effet c'est plus adapté :-) j'ai corrigé

@colinux colinux enabled auto-merge June 5, 2025 08:00
@E-L-T E-L-T force-pushed the fix-dossiers-count-bug-in-manager branch from c87cdb8 to 0bdbe8d Compare June 5, 2025 08:21
@colinux colinux force-pushed the fix-dossiers-count-bug-in-manager branch from 0bdbe8d to 60f2289 Compare June 5, 2025 11:36
@E-L-T E-L-T force-pushed the fix-dossiers-count-bug-in-manager branch from 60f2289 to 3a41243 Compare June 5, 2025 11:56
@E-L-T E-L-T force-pushed the fix-dossiers-count-bug-in-manager branch from 3a41243 to d6d691b Compare June 5, 2025 11:57
@colinux colinux added this pull request to the merge queue Jun 5, 2025
Merged via the queue into main with commit fb6cb8c Jun 5, 2025
27 of 29 checks passed
@colinux colinux deleted the fix-dossiers-count-bug-in-manager branch June 5, 2025 12:30
@github-project-automation github-project-automation bot moved this to En cours in The Kanban Jun 5, 2025
@mfo mfo moved this from En cours to A Communiquer in The Kanban Jun 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: A Communiquer
Development

Successfully merging this pull request may close these issues.

2 participants