[Cashtab][Alias] Status check upon alias registration input pause
ClosedPublic
Actions

Authored by emack on Dec 1 2023, 07:09.

Details

Reviewers

bytesofman
Fabien

Group Reviewers

Restricted Project

Commits

rABCd65b0e51d066: [Cashtab][Alias] Status check upon alias registration input pause

Summary

As per UAT feedback in T3363, this adds an event listener to the aliasName input field so that when the user stops typing for at least 1 second it triggers an API call to check the alias availability. If the alias is unavailable, display error message and disable buttons.

This does not include pending registrations because as per spec users should still be able to submit them should they wish to compete for the registration result.

Test Plan

Navigate to the Alias component with no input into the Alias field and observe no keystroke event listener triggered via the dev console
Input an already registered alias (e.g. watermelon) and ensure event listener does not trigger the API call whilst typing however it triggers the API call once you've stopped typing for 1 second, which results in a "This alias is already owned by ecash:qz..., please try another alias" UI error message. Ensure buttons are disabled upon this.
Input an unregistered alias and ensure the buttons are enabled, with no errors displayed.
Simulate an alias-server outage by malforming the 'alias' endpoint, and observe the "Error retrieving alias details" UI error when you pause an alias input for 1 second and disabling of buttons.

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

emack created this revision.Dec 1 2023, 07:09

Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 1 2023, 07:09

emack requested review of this revision.Dec 1 2023, 07:09

emack edited the summary of this revision. (Show Details)Dec 1 2023, 07:09

Harbormaster completed remote builds in B25784: Diff 43337.Dec 1 2023, 07:12

That's smart, but what is the point of the check button then ? Can it be removed ?

Fabien mentioned this in D14886: [Cashtab][Alias] Update notification icon upon unavailable result from Check Alias button.Dec 1 2023, 09:24

In D14885#334242, @Fabien wrote:

That's smart, but what is the point of the check button then ? Can it be removed ?

Yea the Check button can be removed as this is a more elegant solution. I'll update this diff to also render a UI message when the alias is available for consistency.
I'll deprecate the check button in a separate diff.

In D14885#334256, @emack wrote:

In D14885#334242, @Fabien wrote:

That's smart, but what is the point of the check button then ? Can it be removed ?

Yea the Check button can be removed as this is a more elegant solution. I'll update this diff to also render a UI message when the alias is available for consistency.
I'll deprecate the check button in a separate diff.

After reviewing, there is no need for a specific 'alias is available' message upon this API call because unless it is unavailable, the alias length and registration price will be displayed which is intuitive enough to the user that it is ok to proceed.

As above, will deprecate Check Alias button in another diff.

This does not include pending registrations because as per spec users should still be able to submit them should they wish to compete for the registration result.

Still useful to know there are pending registrations though. The user still gets a popup msg / warning in this case on trying to register?

cashtab/src/components/Alias/Alias.js
188 ↗	(On Diff #43337)	let's give `1000` some kind of variable assignment so it isn't a magic number `const KEY_UP_TIMEOUT_MS = 1000`

This revision now requires changes to proceed.Dec 1 2023, 15:22

In D14885#334392, @bytesofman wrote:

Still useful to know there are pending registrations though. The user still gets a popup msg / warning in this case on trying to register?

yup the confirmation modal via preparePreviewModal already provides the pending warnings which is the next step in this workflow anyway, hence I don't think we need to duplicate it here.

Moved key up timeout to aliasSettings config.

Harbormaster completed remote builds in B25804: Diff 43361.Dec 2 2023, 00:09

Fabien accepted this revision.Dec 3 2023, 11:18

bytesofman requested changes to this revision.Dec 4 2023, 17:06

bytesofman added inline comments.

cashtab/src/components/Alias/Alias.js
157 ↗	(On Diff #43361)
161 ↗	(On Diff #43361)	won't `timeout` always be `undefined` here? Please implement appropriate way to cleanup `setTimeout` in a `useEffect`, e.g. https://stackoverflow.com/questions/53090432/react-hooks-right-way-to-clear-timeouts-and-intervals