Skip to content
This repository has been archived by the owner on Jan 30, 2024. It is now read-only.

bogue: [m-phonefield] Les numéros 1 800 canadiens sont reconnus comme étant au É-U #332

Open
jipigi opened this issue May 26, 2020 · 0 comments
Labels
breaking change Fix or feature that would cause existing functionality to not work as expected bug fix A non-breaking change which fixes an issue

Comments

@jipigi
Copy link
Contributor

jipigi commented May 26, 2020

Description du bogue / Describe the bug
Le numéro de la DGPC 1 877 785-2825 (et tous les autres 1 800 des pays du NANPA) vont déclencher la sélection du drapeau US. En fait, les numéros 1 800 sont partagés entre tous les pays du NANPA, et libphonenumber ne fait pas l'association entre le pays d'appartenance et le no : https://issuetracker.google.com/issues/77836767.

Pour éviter ce problème, il faudrait désactiver l'autodétection lorsque la valeur est modifiée dans le champ. De toute façon, depuis que l'indicatif international est préfixé et non éditable (#320), cette détection ne faisait plus vraiment de sens (À valider avec AEL).

Il faudrait garder la détection lorsque le numéro est assigné par programmation au model pour permettre à AEL de continuer de fonctionner. On pourrait cependant les avertir que la détection pour les 1800 est non fonctionnelle.

Reproduction / To Reproduce

  1. Dans le storybook Detect Country du m-phonefield
  2. Saisir le numéro de la DGFC 1 877 785 2825
  3. Le drapeau US est sélecitonné

Comportement attendu / Expected behavior
Le drapeau canadien reste sélectionné

Captures d'écran / Screenshots
https://imgur.com/5U1ZYQY

Versions affectées / Affected Versions

  • Package: modul-components
  • version: 1.1.9

Liens vers les récits Jira
https://jira.dti.ulaval.ca/browse/ENA2-11218

@jipigi jipigi changed the title bogue: bogue: [m-phonefield] Les numéros 1 800 canadiens sont reconnus comme étant au É-U May 26, 2020
@jipigi jipigi added breaking change Fix or feature that would cause existing functionality to not work as expected bug fix A non-breaking change which fixes an issue labels May 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
breaking change Fix or feature that would cause existing functionality to not work as expected bug fix A non-breaking change which fixes an issue
Projects
None yet
Development

No branches or pull requests

1 participant