-
Notifications
You must be signed in to change notification settings - Fork 76
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
💫 Change geotrek.core
verbose name to "Topology"
#3990
base: master
Are you sure you want to change the base?
Conversation
c62c0f8
to
39ae791
Compare
De mémoire il y a différentes tables de la BDD qui sont préfixées par "core", cela va changer ? Dans l'AdminSite ? |
Passing run #8325 ↗︎
Details:
Review all test suite changes for PR #3990 ↗︎ |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3990 +/- ##
=======================================
Coverage 98.37% 98.37%
=======================================
Files 296 296
Lines 22560 22560
=======================================
Hits 22193 22193
Misses 367 367 ☔ View full report in Codecov by Sentry. |
@camillemonchicourt Les tables et les URL ne vont pas changer, seul le texte va changer (dans l'adminsite, il n'y aura plus marqué "Coeur" mais "Topologie", meme si core_path reste core_path, et que /admin/core/ reste /admin/core/) Le nom des modules est utilisé dans le frontend pour, par exemple, grouper les couches leaflet, et "Coeur" n'y a aucun sens Dans un prochain temps, on pourrait discuter de ce module 'core', car il rassemble les tronçons, les topologies mais aussi les sentiers. Je pense qu'il pourrait changer en 'topologies', et les sentiers pourraient être déplacés de module... mais c'est à discuter |
geotrek.core
verbose name to "Topology"
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.
I suggest to run makemessages and fix translations
docs/changelog.rst
Outdated
@@ -10,6 +10,7 @@ CHANGELOG | |||
- Add `include_externals` filter to Cirkwi trek exports, to allow excluding treks with an external id (eid) (#3947) | |||
- Tourism : add price to TouristicEvent model - ref #3587 | |||
- Add `check_versions` command to check Geotrek, Python, Django, PostgreSQL and PostGIS versions. | |||
- Change core verbose_name to topology |
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.
- Change core verbose_name to topology | |
- Change `geotrek.core` verbose name to "Topology" |
39ae791
to
f963617
Compare
OK je comprends. En effet "cœur" a peu de sens et peu compréhensible. Cependant en regardant ce qu'il y a dans "cœur" dans l'Adminsite (Catégories de sentier, Conforts, Enjeux d'entretien, Labellisations, Réseaux, Sources des tronçons, Statuts de labellisation, Usages), à ma compréhension cela ne correspond pas tant que ça aux "topologies" (qui sont tous les objets gérés en segmentation dynamique incluant les itinéraires, les aménagements, la signalétique...) mais uniquement les tronçons et sentiers. |
a75672e
to
c7b3925
Compare
Le verbose name "Core" dans l'application n'est pas très parlant en français (Cœur). Solution le renomer en Topology ce qui donne "Topologie".
c7b3925
to
eb299da
Compare
Le verbose name "Core" dans l'application n'est pas très parlant en français (Cœur).
Solution le renomer en Topology ce qui donne "Topologie".
Description
Related Issue
Checklist