fix(nms): Display correct axios errors #13986
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Closes #13948.
getErrorMessage
to useelse if
instead ofif
when checking if the error is an instance ofError
. Otherwise, when an axios error message is encountered, theerrorMessage
always gets overwritten to be the default.error
field withmessage
throughoutnms/server
so that it corresponds to the field that gets accessed when returning the axios error message.isAxiosError
function withaxios.isAxiosError
.Test Plan
On master, when trying to create two organizations with the same name on the host site, one gets "Request failed with status code 409". With these changes, one gets "Organization exists already".
Similarly, when trying to create two users with the same name, one gets "Request failed with status code 400" on master, but e.g. "Error: admin@magma.test already exists" with this PR.