Skip to content

Commit e8f5220

Browse files
BrunoQuaresmaEmyrk
authored andcommitted
Fix issues
1 parent d1fba3a commit e8f5220

File tree

4 files changed

+7
-12
lines changed

4 files changed

+7
-12
lines changed

.vscode/settings.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -211,5 +211,5 @@
211211
"go.testFlags": ["-short", "-coverpkg=./..."],
212212
// We often use a version of TypeScript that's ahead of the version shipped
213213
// with VS Code.
214-
"typescript.tsdk": "./site/node_modules/typescript/lib",
214+
"typescript.tsdk": "./site/node_modules/typescript/lib"
215215
}

coderd/userauth.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -833,7 +833,7 @@ func (api *API) userOIDC(rw http.ResponseWriter, r *http.Request) {
833833
// Convert the []interface{} we get to a []string.
834834
groupsInterface, ok := groupsRaw.([]interface{})
835835
if ok {
836-
logger.Debug(ctx, "groups returned in oidc claims",
836+
api.Logger.Debug(ctx, "groups returned in oidc claims",
837837
slog.F("len", len(groupsInterface)),
838838
slog.F("groups", groupsInterface),
839839
)
@@ -854,7 +854,7 @@ func (api *API) userOIDC(rw http.ResponseWriter, r *http.Request) {
854854
groups = append(groups, group)
855855
}
856856
} else {
857-
logger.Debug(ctx, "groups field was an unknown type",
857+
api.Logger.Debug(ctx, "groups field was an unknown type",
858858
slog.F("type", fmt.Sprintf("%T", groupsRaw)),
859859
)
860860
}

site/src/components/SettingsAccountForm/SettingsAccountForm.test.tsx

+2-2
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ describe("AccountForm", () => {
3131
const el = await screen.findByLabelText("Username")
3232
expect(el).toBeEnabled()
3333
const btn = await screen.findByRole("button", {
34-
name: /Update settings/i,
34+
name: /Update account/i,
3535
})
3636
expect(btn).toBeEnabled()
3737
})
@@ -61,7 +61,7 @@ describe("AccountForm", () => {
6161
const el = await screen.findByLabelText("Username")
6262
expect(el).toBeDisabled()
6363
const btn = await screen.findByRole("button", {
64-
name: /Update settings/i,
64+
name: /Update account/i,
6565
})
6666
expect(btn).toBeDisabled()
6767
})

site/src/components/SettingsSecurityForm/SettingsSecurityForm.tsx

+2-7
Original file line numberDiff line numberDiff line change
@@ -64,10 +64,7 @@ export const SecurityForm: FC<SecurityFormProps> = ({
6464
validationSchema,
6565
onSubmit,
6666
})
67-
const getFieldHelpers = getFormHelpers<SecurityFormValues>(
68-
form,
69-
error,
70-
)
67+
const getFieldHelpers = getFormHelpers<SecurityFormValues>(form, error)
7168

7269
if (disabled) {
7370
return (
@@ -81,9 +78,7 @@ export const SecurityForm: FC<SecurityFormProps> = ({
8178
<>
8279
<Form onSubmit={form.handleSubmit}>
8380
<FormFields>
84-
{Boolean(error) && (
85-
<ErrorAlert error={error} />
86-
)}
81+
{Boolean(error) && <ErrorAlert error={error} />}
8782
<TextField
8883
{...getFieldHelpers("old_password")}
8984
autoComplete="old_password"

0 commit comments

Comments
 (0)