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

[m-phonefield] - Remettre code de pays comme préfix #320

Merged
merged 5 commits into from
May 21, 2020

Conversation

matur367
Copy link
Contributor

Description

Mettre le code de pays comme préfix pour améliorer l’expérience utilisateur :)

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)
  • 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)
  • Storybook
  • Test manuel / Sandboxes
  • Autre

Inclure cette section dans les release notes

Liens internes

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

@matur367 matur367 added the enhancement non-breaking change which adds functionality such as new props, slots, event or method to service label May 18, 2020
@matur367 matur367 added this to the 1.1.9 milestone May 18, 2020
@matur367
Copy link
Contributor Author

J'ai changer le code de pays de Saint-Martin, un oublie dans cette PR : https://jira.dti.ulaval.ca/browse/ENA2-11118

Cela permet qu'il n'ai plus de pays avec un préfixe de 4 chiffres et évite le problème suivant : https://jira.dti.ulaval.ca/browse/MODUL-1104

Copy link
Contributor

@jipigi jipigi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Enfin, le remplissage automatique va pouvoir fonctionner correctement en ayant le + et l'indicatif internationnal dans le préfix. Ça va être pas mal plus simple pour l'utilisateur de saisir from scratch car ça ressemble pas mal plus à la saisie d'un numéro national comme on est habitué 👍

J'ai testé la reconnaissance du pays en fonction du numéro fourni au model, et fait le tour des autres storybook, aucune régression, ça marche très bien, GJ!

@matur367 matur367 changed the title [m-phonfield] - Remettre code de pays comme préfix [m-phonefield] - Remettre code de pays comme préfix May 19, 2020
@chuckmah chuckmah merged commit 080307e into master May 21, 2020
@chuckmah chuckmah deleted the feature/ENA2-11118_Mettre_plus_phonefield_prefix branch May 21, 2020 14:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement non-breaking change which adds functionality such as new props, slots, event or method to service
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants