Inquirer icon indicating copy to clipboard operation
Inquirer copied to clipboard
verified publisher icon afucher

Included condition to allow @ character on console Input

Open c3sxr opened this issue 5 years ago • 7 comments

If the digit is D2 (@ special character) then it will go back one character and the next will replace it I included a condition to avoid this and include the @ special character as an input character

c3sxr avatar Oct 02 '20 01:10 c3sxr

hey @c3sxr , could you please add some tests for this bug?

afucher avatar Oct 14 '20 23:10 afucher

Hi @afucher ! I made another push to the forked branch, I included a new class for the email test, applied the changes and returned the version back. Let me know if there's something missing, I'll be glad to help 👍 Thank you!

c3sxr avatar Oct 14 '20 23:10 c3sxr

Hey @c3sxr for some reason it is not reflecting any new changes here in the Pull Request 😕

afucher avatar Oct 14 '20 23:10 afucher

Can you please look again, the File changes tab is showing my new changes image Thanks!

c3sxr avatar Oct 15 '20 00:10 c3sxr

Oh those files I saw, my comment was regarding adding a new test that cover the @ bug

afucher avatar Oct 15 '20 01:10 afucher

Oh you mean on the InquirerUnitTest? Sure let me include it

c3sxr avatar Oct 16 '20 00:10 c3sxr

Can you please take a look and let me know if that looks right?

c3sxr avatar Oct 16 '20 01:10 c3sxr