Skip to content

Commit 7bc98c2

Browse files
refactor(site): do not display error details as message (#9755)
1 parent e26bb2d commit 7bc98c2

File tree

2 files changed

+6
-10
lines changed

2 files changed

+6
-10
lines changed

site/src/api/errors.ts

+1-6
Original file line numberDiff line numberDiff line change
@@ -61,12 +61,7 @@ export const mapApiErrorToFieldErrors = (
6161
export const getErrorMessage = (
6262
error: unknown,
6363
defaultMessage: string,
64-
): string =>
65-
isApiError(error)
66-
? error.response.data.message
67-
: error instanceof Error
68-
? error.message
69-
: defaultMessage;
64+
): string => (isApiError(error) ? error.response.data.message : defaultMessage);
7065

7166
/**
7267
*

site/src/pages/LoginPage/LoginPageView.stories.tsx

+5-4
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { action } from "@storybook/addon-actions";
2-
import { MockAuthMethods } from "testHelpers/entities";
2+
import { MockAuthMethods, mockApiError } from "testHelpers/entities";
33
import { LoginPageView } from "./LoginPageView";
44
import type { Meta, StoryObj } from "@storybook/react";
55

@@ -24,14 +24,15 @@ export const Example: Story = {
2424
},
2525
};
2626

27-
const err = new Error("Username or email are wrong.");
28-
2927
export const AuthError: Story = {
3028
args: {
3129
isLoading: false,
3230
onSignIn: action("onSignIn"),
3331
context: {
34-
error: err,
32+
error: mockApiError({
33+
message: "User or password is incorrect",
34+
detail: "Please, try again",
35+
}),
3536
data: {
3637
authMethods: MockAuthMethods,
3738
hasFirstUser: false,

0 commit comments

Comments
 (0)