Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: add request media #1811

Merged
merged 1 commit into from
Dec 31, 2024
Merged

feat: add request media #1811

merged 1 commit into from
Dec 31, 2024

Conversation

manuel-rw
Copy link
Member

@manuel-rw manuel-rw commented Dec 30, 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)

See #1047

request-series
request-movie
request-search

@manuel-rw manuel-rw self-assigned this Dec 30, 2024
@manuel-rw manuel-rw added the enhancement New feature or request label Dec 30, 2024
Copy link

deepsource-io bot commented Dec 30, 2024

Here's the code health analysis summary for commits e53b06e..0f70968. 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.

Copy link

github-actions bot commented Dec 30, 2024

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 21.5% 7892 / 36698
🔵 Statements 21.5% 7892 / 36698
🔵 Functions 24.61% 285 / 1158
🔵 Branches 61.67% 861 / 1396
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
packages/api/src/router/search-engine/search-engine-router.ts 28.81% 100% 100% 28.81% 13-30, 33-56, 59-71, 77-78, 84-85, 91-100, 106-127, 133
packages/definitions/src/docs/homarr-docs-sitemap.ts 0% 0% 0% 0%
packages/integrations/src/base/searchable-integration.ts 0% 0% 0% 0%
packages/integrations/src/overseerr/overseerr-integration.ts 35.8% 100% 0% 35.8% 23-43, 46-53, 62-80, 83-95, 99-151, 154-160, 163-176, 179-194, 197-212, 215-238, 241-248, 348-351, 378-381, 386-392, 395-406
packages/modals-collection/src/index.ts 0% 0% 0% 0% 1-4
packages/modals-collection/src/search-engines/index.ts 0% 0% 0% 0% 1
packages/modals-collection/src/search-engines/request-media-modal.tsx 0% 0% 0% 0% 1-113
packages/spotlight/src/components/spotlight.tsx 0% 0% 0% 0% 1-170
packages/spotlight/src/components/actions/children-actions.tsx 0% 0% 0% 0% 1-28
packages/spotlight/src/components/actions/items/children-action-item.tsx 0% 0% 0% 0% 1-41
packages/spotlight/src/lib/children.ts 0% 100% 100% 0% 20-27
packages/spotlight/src/modes/external/search-engines-search-group.tsx 0% 0% 0% 0% 1-275
packages/validation/src/common.ts 100% 100% 100% 100%
Generated in workflow #4421 for commit 0f70968 by the Vitest Coverage Report Action

@manuel-rw manuel-rw force-pushed the feat/add-request-media branch 9 times, most recently from 42f5660 to b1c9511 Compare December 30, 2024 18:12
@manuel-rw manuel-rw marked this pull request as ready for review December 30, 2024 18:17
@manuel-rw manuel-rw requested a review from a team as a code owner December 30, 2024 18:17
@manuel-rw manuel-rw force-pushed the feat/add-request-media branch 2 times, most recently from a2036df to 84ff549 Compare December 30, 2024 21:16
Meierschlumpf
Meierschlumpf previously approved these changes Dec 31, 2024
@manuel-rw manuel-rw merged commit 91e1cef into dev Dec 31, 2024
13 checks passed
@manuel-rw manuel-rw deleted the feat/add-request-media branch December 31, 2024 10:05
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants