-
Notifications
You must be signed in to change notification settings - Fork 80
Fix english label for sensitive area shapefile parsers #5137
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
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #5137 +/- ##
=======================================
Coverage 98.47% 98.47%
=======================================
Files 272 272
Lines 22269 22269
=======================================
Hits 21929 21929
Misses 340 340 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Geotrek-admin
|
||||||||||||||||||||||||||||
| Project |
Geotrek-admin
|
| Branch Review |
refs/pull/5137/merge
|
| Run status |
|
| Run duration | 02m 13s |
| Commit |
|
| Committer | Fred C |
| View all properties for this run ↗︎ | |
| Test results | |
|---|---|
|
|
0
|
|
|
0
|
|
|
0
|
|
|
0
|
|
|
22
|
| View all changes introduced in this branch ↗︎ | |
|
Les fichiers de traduction ne semblent pas à jour sur ma branche et le CI échoue (mais ma PR n'est normalement pas concernée). En effet, la mise à jour des fichiers de traduction ajoute 2 nouvelles traduction. Dois-je les ajouter à la PR ? msgid "Log in"
msgstr ""
msgid "Forgotten your password or username?"
msgstr ""
... |
a5fe5c4 to
9f4123d
Compare
|
@lpofredc Hello. Pour les traductions, c'est autre chose, un changement dans la CI ne t'autorise pas à lancer le job de vérification des traductions. Faut que je creuse car ça a été mis en place pour justement ajouter les trads manquantes en cas d'oubli. Mais ça ne fonctionne apparemment que pour les membres de l'orga GeotrekCE. Faut que je vois pour autoriser aussi les forks, quitte à obliger les utilisateurs à les mettres à jour manuellement. Par contre, peux-tu ajouter une entrée dans le changelog pour ta modif ? Si tu mets le changelog, dans la mesure ou les autres tests passent, je mergerai ta PR |
Description
Huge fix!
Related Issue
None
Checklist