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

Fix alternate versions being played from incorrect positions #5373

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

SourSulfur
Copy link

@SourSulfur SourSulfur commented Apr 5, 2024

Changes

Updated current item based on selected media source ID.

Issues
#5372

Copy link

sonarcloud bot commented Apr 5, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@SourSulfur
Copy link
Author

I replaced the entire current item with the new item fetched using the selected media source's ID. To fix this issue specifically, I believe just replacing the User Data should be enough but I noticed that the item's name/ID was inconsistent too - so I just replaced the entire thing.

Could use some feedback on this and if updating only the User Data and name is enough, or if the approach itself could be improved.

@SourSulfur SourSulfur marked this pull request as ready for review April 5, 2024 17:01
@SourSulfur SourSulfur requested a review from a team as a code owner April 5, 2024 17:01
@thornbill thornbill added this to the v10.10.0 milestone Apr 23, 2024
@thornbill thornbill added enhancement Improve existing functionality or small fixes ui & ux This PR or issue mainly concerns UI & UX discussion needed Requires input from the community labels May 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion needed Requires input from the community enhancement Improve existing functionality or small fixes ui & ux This PR or issue mainly concerns UI & UX
Projects
Status: In progress
Development

Successfully merging this pull request may close these issues.

None yet

2 participants