-
Notifications
You must be signed in to change notification settings - Fork 91
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
ETQ admin, dans la page "toutes les démarches", je veux que les tags soient harmonisés #10876
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #10876 +/- ##
==========================================
- Coverage 84.51% 84.50% -0.01%
==========================================
Files 1136 1140 +4
Lines 25207 25244 +37
Branches 4718 4724 +6
==========================================
+ Hits 21303 21332 +29
- Misses 3904 3912 +8 ☔ View full report in Codecov by Sentry. |
b2cd4df
to
205504e
Compare
f7ffc48
to
c263842
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kara22 j'ai creusé cette histoire de CI qui soi-disant ne passait pas par le controller qui filtrait les tags. Et j'ai eu raison, car effectivement les tests ne passent pas par les filtres et ya un bug ! J'ai rajouté un test qui échoue pour le démontrer : toutes les procédures sont renvoyées, on ne filtre rien. Je te laisserai faire le fix ;)
J'ai aussi rajouté 2 petits commits annexes qui améliorent très légèrement sur le combobox et le wording (indépendants de cette PR, mais ça se voit dans ce contexte).
5eaf103
to
ec472b2
Compare
…éfinie dans la page toutes les démarches
#10333