-
Notifications
You must be signed in to change notification settings - Fork 58
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
AUTH UI - Improve error message when affiliating an already-affiliated NR #11333
Comments
@sienna-blumstengel Hey I remember talking about this exact use case back when we were designing this! Question - I thought we (Relationships) built the dashboard so it could affiliate a business to multiple accounts - if the same wasn't done for Name Requests we should do that. I can look at re-wording the dialog though - it would still be applicable if the NR was used to incorporate/register - and it is confusing |
I think there might be some flow / race issues to work out if multiple accounts have an NR. |
@thorwolpert @forgeuxGH5 since we already have a work-around (the first account to release the name); then I suggest we focus on fixing the root of the problem - allowing an NR to be affiliated to multiple accounts. @sienna-blumstengel could you create an NR on one account and then try affiliating it to another test account to verify this is indeed the issue? Can you also communicate to staff this workaround? @jyoti3286 - can you check with your team if this was a missed requirement? And if so, can you create a ticket? Either your team or SRE team could fix this? not sure when auth stuff transitioning so also copying @jinghualicgi |
That'll be a bigger fix, as the downstream apps that consume it in draft, will need to do more checks that another account hasn't used it. |
Or as a 2 stage fix. |
Here's some text for the dialogs: Use case: Name request managed by another account (not consumed) Use case Name request used (consumed) |
We should probably have more elegant handling of used name requests (e.g., add the business) but that's a bit more involved and would need a bit more work. Thinking about the multiple account scenarios a bit, we probably need to look at that more in addition to the things @thorwolpert mentioned, for example, things like receipt and refund handling across accounts. |
This isn't applicable anymore, I think consumed doesn't even show up in the search |
Description
This issue was reported in ops incident INC0146295.
A client was trying to add an NR to their account, but the following error message would show.
The NR was NOT consumed, it was still in approved status.
Everyone was confused until Sienna remembered that this is a known issue. This error message shows when a client tries to affiliate an NR to their account that has already been affiliated to someone else's account. Because we use the term "Consumed" for actually using the NR in a filing, this message is very confusing.
TO DO
Change the error message to the following:
The error message should show whenever a client tries to affiliate an NR to their account that is already affiliated to someone else's account.
The text was updated successfully, but these errors were encountered: