Skip to content
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

feat: add api keys #991

Merged
merged 2 commits into from
Oct 5, 2024
Merged

feat: add api keys #991

merged 2 commits into from
Oct 5, 2024

Conversation

manuel-rw
Copy link
Member

@manuel-rw manuel-rw commented Aug 16, 2024


Homarr

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

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

Copy link

deepsource-io bot commented Aug 16, 2024

Here's the code health analysis summary for commits 41538a7..c549121. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript✅ SuccessView Check ↗

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

@manuel-rw manuel-rw force-pushed the feat/implement-openapi branch 6 times, most recently from 5191d4e to 482c490 Compare August 25, 2024 15:59
Base automatically changed from feat/implement-openapi to dev August 25, 2024 16:03
@manuel-rw manuel-rw force-pushed the feat/add-api-keys branch 2 times, most recently from 32ff799 to b3e6697 Compare September 1, 2024 14:35
Copy link

github-actions bot commented Sep 1, 2024

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 38.7% 14256 / 36830
🔵 Statements 38.7% 14256 / 36830
🔵 Functions 34.79% 365 / 1049
🔵 Branches 70.28% 1261 / 1794
File Coverage
File Stmts % Branch % Funcs % Lines Uncovered Lines
Changed Files
apps/nextjs/src/app/[locale]/manage/tools/api/page.tsx 0% 0% 0% 0% 1-2, 4-7, 9-11, 13-14, 16-19, 21-24, 26-40, 42
apps/nextjs/src/app/[locale]/manage/tools/api/components/api-keys.tsx 0% 0% 0% 0% 1, 3-4, 6, 9-13, 15, 21-31, 33-46, 48-51, 53-61, 63-64, 66-70, 72-76, 78
apps/nextjs/src/app/[locale]/manage/tools/api/components/copy-api-key-modal.tsx 0% 0% 0% 0% 1-2, 4-5, 7-22, 24-25, 27-28, 30-34
apps/nextjs/src/app/[locale]/manage/tools/api/components/swagger-ui.tsx 0% 0% 0% 0% 1, 4, 7-9, 15, 17-20, 22-23
apps/nextjs/src/app/api/[...trpc]/route.ts 0% 0% 0% 0% 1, 3, 5-8, 10-12, 14-20, 22-25, 27-29, 31-48, 50-53, 55-57
packages/api/src/open-api.ts 0% 0% 0% 0% 1, 3, 5-19
packages/api/src/root.ts 0% 0% 0% 0% 1-17, 19-36
packages/api/src/router/apiKeys.ts 0% 0% 0% 0% 1-4, 6, 8-41
packages/api/src/router/app.ts 90.71% 100% 100% 90.71% 44-48, 70-77
packages/api/src/router/user.ts 47.21% 78.94% 100% 47.21% 77-78, 80, 96-101, 103-110, 112-117, 119-124, 126-131, 148-156, 159-165, 175-188, 191-202, 204-209, 211, 220-224, 227-231, 249, 251-256, 258-266, 268-273, 275-280, 283, 285-287, 289-291, 293-299, 301-308, 313, 315-320, 322-327, 329-334, 336-341, 344-349, 383-387
packages/auth/callbacks.ts 85.55% 100% 75% 85.55% 35-47
packages/auth/server.ts 0% 0% 0% 0% 1
packages/db/schema/mysql.ts 100% 100% 100% 100%
packages/db/schema/sqlite.ts 100% 100% 100% 100%
packages/translation/src/lang/en.ts 100% 100% 100% 100%
Generated in workflow #3164 for commit c549121 by the Vitest Coverage Report Action

@manuel-rw manuel-rw force-pushed the feat/add-api-keys branch 10 times, most recently from 6ea0a20 to 377dbc4 Compare September 6, 2024 20:52
@manuel-rw manuel-rw force-pushed the feat/add-api-keys branch 9 times, most recently from 2f0ab46 to 6928d90 Compare September 25, 2024 18:04
@manuel-rw manuel-rw marked this pull request as ready for review September 25, 2024 18:08
@manuel-rw manuel-rw requested a review from a team as a code owner September 25, 2024 18:08
@manuel-rw manuel-rw self-assigned this Sep 25, 2024
@manuel-rw manuel-rw added the enhancement New feature or request label Sep 25, 2024
@manuel-rw manuel-rw force-pushed the feat/add-api-keys branch 2 times, most recently from e4769d7 to df2f30b Compare September 25, 2024 18:12
@Meierschlumpf Meierschlumpf merged commit b14f82b into dev Oct 5, 2024
10 checks passed
@Meierschlumpf Meierschlumpf deleted the feat/add-api-keys branch October 5, 2024 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants