Update admin-structures-list.component.ts#4197
Conversation
Précision sur le label qui n'est pas adapté : les structures actives ne figurent pas forcément dans cette catégorie. Remplacement du terme Actives par Validées qui est plus adapté et permet d'être cohérent par rapport au statut "non validées"
revu-bot
left a comment
There was a problem hiding this comment.
Summary
This PR renames the "Actives" tab label to "Validées" in the admin structures list component. The change is semantically correct — structures with a VALIDE status are not necessarily active, and aligning the label with the VALIDE key and the existing "Non validées" label improves consistency.
| File | Lines | Severity | Issue |
|---|---|---|---|
admin-structures-list.component.ts |
236 | MINOR | No issues — the label change is accurate and consistent |
The change is minimal, targeted, and correct. No bugs, security issues, or architectural concerns are introduced. The label now accurately reflects the underlying VALIDE status key and is consistent with the "Non validées" counterpart already present in the tab list.
No actionable review comments are required for this PR.
|
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #4197 +/- ##
========================================
Coverage 62.12% 62.12%
========================================
Files 1026 1026
Lines 20890 20890
Branches 4068 4092 +24
========================================
Hits 12977 12977
+ Misses 7473 7269 -204
- Partials 440 644 +204 see 78 files with indirect coverage changes Continue to review full report in Codecov by Harness.
🚀 New features to boost your workflow:
|



Précision sur le label qui n'est pas adapté : les structures actives ne figurent pas forcément dans cette catégorie. Remplacement du terme Actives par Validées qui est plus adapté et permet d'être cohérent par rapport au statut "non validées"