entrouvert
/
hobo
16
0
Fork 0
Code Issues Pull Requests 16 Releases Activity

manage Publik's phone-related settings through dedicated form (#72760) #15

Merged
pmarillonnet merged 1 commits from wip/72760-sms-default-country-code-selection into main 2023-05-30 09:24:31 +02:00
Conversation 3 Commits 1 Files Changed 6 +71 -3
pmarillonnet commented 2023-02-03 15:43:16 +01:00
Owner
Copy Link

Il y avait dans les discussions directement dans le ticket lié des choses relatives au mauvais nommage des entrées dans PHONE_COUNTRY_CODES, et de DEFAULT_COUNTRY_CODE qui devrait en fait s’appeler LOCAL_COUNTRY_CODE. C’est corrigé, il faudra le ticket qui procède au même renommage s/default/local côté authentic, je vais le créer.

Il y a aussi, dans cette nouvelle version du code, la possibilité de charger dans les settings de tenant cette liste de code pays.

Il y avait dans les discussions directement dans le ticket lié des choses relatives au mauvais nommage des entrées dans PHONE_COUNTRY_CODES, et de DEFAULT_COUNTRY_CODE qui devrait en fait s’appeler LOCAL_COUNTRY_CODE. C’est corrigé, il faudra le ticket qui procède au même renommage s/default/local côté authentic, je vais le créer. Il y a aussi, dans cette nouvelle version du code, la possibilité de charger dans les settings de tenant cette liste de code pays.
pmarillonnet added 1 commit 2023-02-03 15:43:17 +01:00
pmarillonnet changed title from manage Publik's phone-related settings through dedicated form (#72760) to WIP: manage Publik's phone-related settings through dedicated form (#72760) 2023-02-03 16:07:53 +01:00
pmarillonnet commented 2023-02-03 16:09:15 +01:00
Author
Owner
Copy Link

Je me rends compte que ce n’est pas la bonne façon de procéder pour la gestion de la variable ´phone_country_codes´ du hobo.json au travers des briques. Je vais revoir ça.

Je me rends compte que ce n’est pas la bonne façon de procéder pour la gestion de la variable ´phone_country_codes´ du hobo.json au travers des briques. Je vais revoir ça.
pmarillonnet force-pushed wip/72760-sms-default-country-code-selection from 1028ea29a7 to 8a5676e39d 2023-02-09 12:33:19 +01:00 Compare
pmarillonnet changed title from WIP: manage Publik's phone-related settings through dedicated form (#72760) to manage Publik's phone-related settings through dedicated form (#72760) 2023-02-09 12:36:07 +01:00
pmarillonnet commented 2023-02-09 12:36:44 +01:00
Author
Owner
Copy Link

Je me rends compte que ce n’est pas la bonne façon de procéder pour la gestion de la variable ´phone_country_codes´ du hobo.json au travers des briques. Je vais revoir ça.

Après réflexion, je pense que ce n’est pas nécessaire de créer explicitement une variable d’environnement juste pour la liste des codes téléphoniques supportés, il faut simplement gérer le cas où elle est définie, qu’elle prenne le dessus sur la config locale à hobo. C’est ce que fait cette nouvelle version de la PR.

> Je me rends compte que ce n’est pas la bonne façon de procéder pour la gestion de la variable ´phone_country_codes´ du hobo.json au travers des briques. Je vais revoir ça. Après réflexion, je pense que ce n’est pas nécessaire de créer explicitement une variable d’environnement juste pour la liste des codes téléphoniques supportés, il faut simplement gérer le cas où elle est définie, qu’elle prenne le dessus sur la config locale à hobo. C’est ce que fait cette nouvelle version de la PR.
fpeters requested changes 2023-02-28 07:17:43 +01:00
hobo/environment/models.py Outdated
@ -52,6 +52,8 @@ AUTO_VARIABLES = (
'meta_keywords',
'sms_url',
'sms_sender',
'phone_country_codes',
fpeters commented 2023-02-28 07:17:38 +01:00
Owner
Copy Link

Ça ne me semble plus nécessaire, phone_country_codes n'est pas enregistré dans envionment_variable, il me semble.

Ça ne me semble plus nécessaire, phone_country_codes n'est pas enregistré dans envionment_variable, il me semble.
fpeters force-pushed wip/72760-sms-default-country-code-selection from 8a5676e39d to 536402e533 2023-05-28 09:04:56 +02:00 Compare
fpeters commented 2023-05-28 09:32:58 +02:00
Owner
Copy Link

Ça ne me semble plus nécessaire, phone_country_codes n'est pas enregistré dans envionment_variable, il me semble.

J'ai poussé un commit prenant compte de ça.

J'ai aussi :

  • un commit pour rendre les champs de l'écran optionnels (parce qu'on veut pouvoir configurer la zone sans avoir de connecteur SMS)
  • un commit pour modifier l'help_text

Avec ça (squashé) la branche me semble ok.

(j'aimerais ça ce cycle pour aller avec l'introduction de la validation "téléphone mobile (local)" pour laquelle il est utile de connaitre la zone "locale").

> Ça ne me semble plus nécessaire, phone_country_codes n'est pas enregistré dans envionment_variable, il me semble. J'ai poussé un commit prenant compte de ça. J'ai aussi : * un commit pour rendre les champs de l'écran optionnels (parce qu'on veut pouvoir configurer la zone sans avoir de connecteur SMS) * un commit pour modifier l'help_text Avec ça (squashé) la branche me semble ok. (j'aimerais ça ce cycle pour aller avec l'introduction de la validation "téléphone mobile (local)" pour laquelle il est utile de connaitre la zone "locale").
fpeters approved these changes 2023-05-30 09:12:59 +02:00
fpeters left a comment
Owner
Copy Link

Avec les modifs suggérées squashées.

Avec les modifs suggérées squashées.
pmarillonnet force-pushed wip/72760-sms-default-country-code-selection from 536402e533 to 99eb6e3f35 2023-05-30 09:17:21 +02:00 Compare
pmarillonnet merged commit 99eb6e3f35 into main 2023-05-30 09:24:31 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
fpeters
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/hobo#15
No description provided.
Delete Branch "wip/72760-sms-default-country-code-selection"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?