[m-postalcodefield] Changé la casse des codes de pays

sioue19 commented 4 years ago

Description

Les valeurs de l'enum de postalcodefield auraient dûs être en majuscules pour être consistents avec les autres codes de pays qu'on retrouve dans Modul (comme dans phonefield, par exemple).

Types de changements

[ ] Correction de bug (sans breaking change)
[ ] Amélioration (ajout par example une nouvelle propriété, évènement, slot ou méthode à un composant existant sans breaking change)
[ ] Nouvelle fonctionalité (nouveau composant, directive, filtre ou service)
[x] Breaking change (modification à une fonctionnalités existante qui nécessite une migration remplir la section release note)
[ ] Refactoring/ménage (sans breaking change)
[ ] Documentation/storybook (changement à la documentation ou aux storybooks qui n'affecte aucun package)
[ ] Autre

Comment cela peut-il être testé?

[ ] Test unitaire (un nouveau test unitaire à été fait)
[x] Storybook
[ ] Test manuel / Sandboxes
[ ] Autre

Inclure cette section dans les release notes

À chaque endroit où le filtre postalCodeFilter est utilisé, il faut changer la casse du paramètre isoCountry afin que la valeur soit en minuscule.

Pour le m-postalcodefield, il faut aussi s'assurer que la casse de la prop soit en minuscule. Dans le cas de Brio, ça ne cause pas de problème parce que c'était les valeurs d'enum qui étaient utilisées. La correction apportée dans Modul n'apporte pas de breaking change.

Liens internes

https://jira.dti.ulaval.ca/browse/ENA2-11208

chuckmah commented 4 years ago

@sioue19 on pourrait rajouter un this.postalCodeFormat.toLowerCase() pour ne pas faire de breaking changes ? qu'en penses-tu ?

sioue19 commented 4 years ago

@chuckmah Effectivement! Bon point!

Je l'ai rajouté dans le switch du postalcodefield, ainsi que dans le filtre.

sioue19 commented 4 years ago

En ajoutant le toLowerCase, je me suis rendu compte que le codepostalfield utilise le format CA par défaut. J'ai mis la même affaire dans le filtre pour être consistant.

Pour que le champ affiche le texte sans modifier le format, il faut vraiment mettre Other.

ulaval / modul