Produits Entr'ouvert » Hobo

Ça poserait settings.DEFAULT_COUNTRY_CODE pour correspondre à ce qu'utilise authentic.

(attention on a désormais hobo dans gitea).

'lang': 'BE'

Ce qui est noté "lang" ici semble être ce qui se trouve au final passé en paramètre dans phonenumbers.parse(), qui s'y appelle plutôt "region". Je serais pour ne pas mélanger les termes.

J'ai aussi globalement une hésitation sur le nommage, je note dans #73948#note-1 que je serais plus à l'aise avec une appellation local-phone-region ("FR") ou local-phone-country-code (33). (oui ça demanderait à modifier l'existant, dans wcs où c'est "FR" qui est tapé, dans combo et authentic où c'est "33" qui est tapé (vs 33 sous forme de nombre si on voulait suivre c qui est attendu par phonenumbers.region_code_for_country_code).

Frédéric Péters a écrit :

(attention on a désormais hobo dans gitea).

(Oui, soit c’est un oubli de ma part soit le patch date d’avant la migration à gitea. La nouvelle version du patch ferait l’objet d’une PR gitea.)

'lang': 'BE'

Ce qui est noté "lang" ici semble être ce qui se trouve au final passé en paramètre dans phonenumbers.parse(), qui s'y appelle plutôt "region". Je serais pour ne pas mélanger les termes.

Ok, à relire mon propre patch un mois plus tard je vois plus problématique encore : le contenu entier de PHONE_COUNTRY_CODES vient d’authentic, il y a une bête duplication sans propagation de cette setting au travers des briques. En l’état, pour le pays par défaut choisi dans hobo, il faut que l’entrée correspondante existe dans le dictionnaire PHONE_COUNTRY_CODES déclaré localement dans authentic pour que ça fonctionne. Peut-être pourrait-on propager cette settings ? Est-ce qu’on aurait aussi intérêt à permettre d’éditer ce dictionnaire dans hobo, en laissant l’administrateur éditer les différentes régions, avec le code pays et l’indicatif téléphonique correspondant ?

J'ai aussi globalement une hésitation sur le nommage, je note dans #73948#note-1 que je serais plus à l'aise avec une appellation local-phone-region ("FR") ou local-phone-country-code (33). (oui ça demanderait à modifier l'existant, dans wcs où c'est "FR" qui est tapé, dans combo et authentic où c'est "33" qui est tapé (vs 33 sous forme de nombre si on voulait suivre c qui est attendu par phonenumbers.region_code_for_country_code).

Ok, oui, il y a sans doute des choses à améliorer de ce côté là, ce sera plus clair une fois qu’on aurait tranché sur le souci de portée des setttings mentionné plus haut.

Paul Marillonnet (pmarillonnet) a ouvert une pull request sur Gitea concernant cette demande :

• URL : https://gitea.entrouvert.org/entrouvert/hobo/pulls/15
• Titre : manage Publik's phone-related settings through dedicated form (#72760)
• Modifications : https://gitea.entrouvert.org/entrouvert/hobo/pulls/15/files

Automatic expiration