Page MenuHomePhabricator

[Cashtab][Alias] Add trimmed input null check after input clearance
ClosedPublic

Authored by emack on Dec 6 2023, 00:46.

Details

Summary

T3363

UAT picked up an edge case where after inputting a valid alias input and then clearing it from the input field, the keyup event will trigger an API call on the-now null value.
This diff prevents the alias API check if the trimmed input value is null or an empty string.

Test Plan

npm test
npm start
enter a valid alias, wait for 1 second, then delete it
ensure no "Error retrieving alias status" pops up on the UI
regression test the recent feature where entering a registered alias (e.g. watermelon) will render an input error on the frontend

Diff Detail

Repository
rABC Bitcoin ABC
Branch
nullAliasCheck
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 25838
Build 51256: Build Diffcashtab-tests
Build 51255: arc lint + arc unit