Skip to content
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

Use locale name fields #400

Open
wants to merge 2 commits into
base: bano_v3
Choose a base branch
from
Open

Conversation

frodrigo
Copy link
Member

En lien avec #397

Ce PR propose de passer des tags->'*_name' au *_name.
Il y a quand même une modification du schéma car une des tables n'a pas ces champs.

@vdct à toi de me dire si tu veux de cette modif ou pas. En ajoutant beaucoup plus de champs name:*, ça devient moins élégant. frodrigo@46ee2e1

@vdct
Copy link
Member

vdct commented Feb 22, 2024

Quitte à changer le schema via le .yml je préfèrerais qu'on en profite pour ajouter comme champs en dur tous les champs des langues régionales histoire de ne pas re-casser le schéma dans un avenir proche si on veut s'en servir. Changer le .yml implique de recharger la BD intégralement, alors autant le faire le moins souvent possible. Est-ce que tu peux augmenter cette PR en conséquence ?

@frodrigo
Copy link
Member Author

Quitte à changer le schema via le .yml je préfèrerais qu'on en profite pour ajouter comme champs en dur tous les champs des langues régionales histoire de ne pas re-casser le schéma dans un avenir proche si on veut s'en servir. Changer le .yml implique de recharger la BD intégralement, alors autant le faire le moins souvent possible. Est-ce que tu peux augmenter cette PR en conséquence ?

Voilà c'est fait.

Je suggère en suivant de prendre aussi cette PR #396 (une fois rebasé)

@vdct
Copy link
Member

vdct commented Mar 8, 2024

Désolé pour le délai
je ne souhaite pas merger dans le même temps la modif du yml et l'utilisation des champs dans les requêtes. Avoir les nouveaux champs en base dans un premier temps va être utile pour analyser le contenu. Je propose de gérer dans une PR séparée la partie consommation, c'est à dire l'inclusion dans les requêtes.
Par ailleurs je trouve dommage de garder la notation OSM façon namespace avec les ":" dans le nom des champs de la base, ça complexifie pour rien l'écriture à coup de guillemets. Je propose de remplacer ":" par "_" : name_oc, name_eu...

@frodrigo
Copy link
Member Author

frodrigo commented Mar 8, 2024

Fait dans #401

@frodrigo frodrigo changed the title Use old_name and alt_name fields Use locale name fields Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants