-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ajout de son nom de naissance #7
Conversation
Possible de le réecrire de la manière du commit ce0e581 ? Merci beaucoup pour ta PR ! |
js/correct.js
Outdated
@@ -7,7 +7,8 @@ while(textNode=walk.nextNode()) { | |||
'Monsieur le Président|'+ | |||
'Président de la République française|'+ | |||
'Président de la République' + |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Il manque le pipe final pour signifier les différentes options, de même qu'a la ligne suivante pour le nom complet de Manu.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
C'est exact, oubli de ma part. C'est corrigé.
Parfait, merci beaucoup ! |
js/correct.js
Outdated
@@ -8,7 +8,7 @@ while(textNode=walk.nextNode()) { | |||
'Président de la République française|'+ | |||
'Président de la République|' + | |||
'Emmanuel Jean-Michel Frédéric Macron|', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Désolé de rouvrir, mais ça marche avec la virgule au lieu du + ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
C'était une bien belle erreur, corrigée dans #11 😄
No description provided.