Additional PR for Issue #653

[danielvydra]

Creating additional pull request after solving created issue #653

This new PR:

• Fixes for issue#653: Fixed enter key handling, removed redundant error message, added some comments, refactoring

Old PR:

• 666

• Solving issue #653, including fixing bug that previously allowed to use and fill in partial/invalid phone number; form adjustment for queue page as well
• Note: This was second PR since forking project not contain latest updates. So fetched upstream project changes, than applied cherry-pick for my changed files. Hope that will work now.

Feel free to review changes @daltonfury42 All should be fine now Hope can be merged

[danielvydra]

@daltonfury42 I am sorry, by mistake I messed up my branch. On the other hand, I made fix and refactoring based on your tips 🙂 Hope will be okay.

[daltonfury42]

No worries. I will review in 10-12 hours.

Before I review, can you run eslint on your branch and fix any leftover errors for changes you made:

npm i -g eslint
cd simplq/
eslint --fix 

[danielvydra]

Sure, I run that - no errors, everything seems to be working fine.

[daltonfury42]

I was expecting it to auto format these white spaces etc, can you check once more?

[danielvydra]

Well, IntelliJ Webstorm automatically add 4-space indent (which is common thing I think) and after running eslint I got "found 0 vulnerabilities". Previously there was only 2-space indent.

[daltonfury42]

@maaverik Sajmal badly wants this feature. I am merging. Please leave any comments if you have, and I or Daniel can take it up as a follow up PR.

@danielvydra Good job on the PR, happy hacking!

[danielvydra]

Cool, thank you! I am new to open source contribution and like this project. 🙂