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.

Sign up for GitHub

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

Jump to bottom

Refactor viewitemcontainer #4691

Merged
merged 2 commits into from
Oct 12, 2023
Merged

Refactor viewitemcontainer #4691

merged 2 commits into from
Oct 12, 2023

Conversation

grafixeyehero
Copy link
Member

No description provided.

@jellyfin-bot
Copy link
Collaborator

This pull request has merge conflicts. Please resolve the conflicts so the PR can be successfully reviewed and merged.

@jellyfin-bot jellyfin-bot added the merge conflict Conflicts prevent merging label Jul 6, 2023
@jellyfin-bot jellyfin-bot removed the merge conflict Conflicts prevent merging label Jul 8, 2023
@sonarcloud
Copy link

sonarcloud bot commented Jul 8, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

No Coverage information No Coverage information
0.0% 0.0% Duplication

@grafixeyehero grafixeyehero marked this pull request as ready for review July 8, 2023 15:33
@grafixeyehero grafixeyehero requested a review from a team as a code owner July 8, 2023 15:33
@grafixeyehero grafixeyehero marked this pull request as draft July 8, 2023 17:15
@jellyfin-bot jellyfin-bot added the merge conflict Conflicts prevent merging label Jul 14, 2023
@jellyfin-bot
Copy link
Collaborator

This pull request has merge conflicts. Please resolve the conflicts so the PR can be successfully reviewed and merged.

@jellyfin-bot jellyfin-bot removed the merge conflict Conflicts prevent merging label Sep 13, 2023
@grafixeyehero grafixeyehero force-pushed the refactor--viewitemcontainer branch 9 times, most recently from 95f8faf to 8b3c2d7 Compare September 17, 2023 21:08
@grafixeyehero grafixeyehero added enhancement Improve existing functionality or small fixes typescript PRs or issues relating to TypeScript code cleanup Cleanup of legacy code or code smells labels Sep 17, 2023
@jellyfin-bot
Copy link
Collaborator

This pull request has merge conflicts. Please resolve the conflicts so the PR can be successfully reviewed and merged.

@jellyfin-bot jellyfin-bot added the merge conflict Conflicts prevent merging label Sep 27, 2023
@jellyfin-bot jellyfin-bot removed the merge conflict Conflicts prevent merging label Oct 2, 2023
@grafixeyehero grafixeyehero mentioned this pull request Oct 2, 2023
Closed
@grafixeyehero grafixeyehero force-pushed the refactor--viewitemcontainer branch 2 times, most recently from c0b17d0 to 3325fe0 Compare October 4, 2023 20:16
@grafixeyehero grafixeyehero marked this pull request as ready for review October 4, 2023 20:26
@thornbill thornbill force-pushed the refactor--viewitemcontainer branch 2 times, most recently from 5fda3d5 to 734f6c1 Compare October 10, 2023 05:04
thornbill
thornbill reviewed Oct 11, 2023
View reviewed changes
src/utils/items.ts Outdated Show resolved Hide resolved
@sonarcloud
Copy link

sonarcloud bot commented Oct 11, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@jellyfin-bot
Copy link
Collaborator

Cloudflare Pages deployment

Latest commit f238b58
Status ✅ Deployed!
Preview URL https://bbc914e2.jellyfin-web.pages.dev
Type 🔀 Preview

View build logs
View bot logs

@thornbill thornbill merged commit 92ceecd into jellyfin:master Oct 12, 2023
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thornbill thornbill thornbill approved these changes

Assignees
No one assigned
Labels
cleanup Cleanup of legacy code or code smells enhancement Improve existing functionality or small fixes typescript PRs or issues relating to TypeScript code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@grafixeyehero @jellyfin-bot @thornbill