profile: format phone numbers at cell-rendering time (#72769) #23
Ghost
Loading…
Reference in New Issue
No description provided.
Delete Branch "wip/72769-profile-cell-phone-number-formatting"
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?
@ -49,0 +52,4 @@def get_formatted_phone(value, country_code=None):if country_code is None:country_code = settings.DEFAULT_COUNTRY_CODEOn devrait quand même poser un « DEFAULT_COUNTRY_CODE = '33" » dans combo/settings.py (pour ne pas avoir de code qui fasse appel à un settings inexistant, combo devrait pouvoir tourner "sans hobo").
@ -32,0 +38,4 @@data['phone'] = '+336a23c5678'requests_get.return_value = mock.Mock(content=json.dumps(data), json=lambda: data, status_code=200)context = cell.get_cell_extra_context({'synchronous': True, 'selected_user': admin_user})assert context['profile_fields']['phone']['value'] == '+336a23c5678'Ajouter un test qui montre qu'un numéro genre en +221 s'affiche bien avec son indicatif international ?
Cf les commentaires ci-dessus
b8901ab4a6to58977db7b4Bien vus, merci. J’ai pris en compte tes remarques dans cette nouvelle version de la PR.
@ -49,0 +73,4 @@if country_code == str(pn.country_code):return phonenumbers.format_number(pn, phonenumbers.PhoneNumberFormat.NATIONAL)return phonenumbers.format_number(pn, phonenumbers.PhoneNumberFormat.E164)phonenumbers.PhoneNumberFormat.INTERNATIONAL serait plus joli (+221 77 643 93 28)
Ok, je n’avais pas réalisé que ces deux formats différaient, my bad. C’est corrigé et c’est mieux ainsi, en effet.
58977db7b4to16e6fc8711Ca me semble ok (même si pour ma part j'aurai préféré qu'on affiche systématiquement le format international).