status-im / ens-usernames

DApp to register usernames for Status Network
MIT License
19 stars 11 forks source link

Show error screen when special characters used in search #9

Closed Serhy closed 5 years ago

Serhy commented 6 years ago

Most of the special chars are not allowed to be searched for supposed ENS name. However some them are. We need to validate 'Search' input field (on all screens where we display it) allowing only letters and numbers be allowed as ENS value.

To reproduce: 1) Navigate to latest END Dapp prototype in testnet (https://ipfs.infura.io/ipfs/QmYLj7gYZfKPee4BirNPBug5z41UbYf699Gy78K9M1x4pi/) 2) Search for a •••...’€€€. Expected result: Names are made with letters and numbers only screen keep stay Actual result: •••...’€€€.statuofus.eth available 3) Now staying on this screen edit ENS name to -/:;()$&@.,?!’()status Expected result: Redirect to Names are made with letters and numbers only screen Actual result: -/:;()$&@.,?!’()status available for registration 4) Hit 'Register with transaction' button Actual result: Endless spinner appears.

screen shot 2018-09-12 at 17 31 57
alwx commented 5 years ago

Fixed already.