fix: it was possible to go to the create user page when credentials login disabled

homarr-labs / homarr

A modern and easy to use dashboard. 14+ integrations. 10K+ icons built in. Authentication out of the box. No YAML, drag and drop configuration.

https://homarr.dev

MIT License

18 stars 5 forks source link

Closed Meierschlumpf closed 1 month ago

Meierschlumpf commented 1 month ago

Thank you for your contribution. Please ensure that your pull request meets the following pull request:

[x] Builds without warnings or errors (pnpm buid, autofix with pnpm format:fix)
[x] Pull request targets dev branch
[x] Commits follow the conventional commits guideline
[x] No shorthand variable names are used (eg. x, y, i or any abbrevation)

deepsource-io[bot] commented 1 month ago

Here's the code health analysis summary for commits 2f3619b..e471db0. View details on DeepSource ↗.

Analyzer	Status	Summary	Link
JavaScript	✅ Success		View Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

github-actions[bot] commented 1 month ago

Status	Category	Percentage	Covered / Total
🔵	Lines	38.95%	13987 / 35910
🔵	Statements	38.95%	13987 / 35910
🔵	Functions	35.14%	363 / 1033
🔵	Branches	70.67%	1253 / 1773

File Coverage

File	Stmts	% Branch	% Funcs	% Lines	Uncovered Lines
Changed Files
apps/nextjs/src/app/[locale]/manage/users/page.tsx	0%	0%	0%	0%	1-3, 5-7, 9-10, 12-15, 17-19, 21-25, 27
apps/nextjs/src/app/[locale]/manage/users/_components/user-list.tsx	0%	0%	0%	0%	1, 3-6, 8, 11-14, 21-26, 28-40, 42-52, 54, 56-59, 61-81, 83-87, 89
apps/nextjs/src/app/[locale]/manage/users/create/page.tsx	0%	0%	0%	0%	1, 3-4, 6-8, 10-11, 13, 15-18, 20-23, 25-29, 31

Generated in workflow #3115 for commit e471db0 by the Vitest Coverage Report Action