Skip to content

Retrieve relevant description#116

Merged
holta merged 1 commit into
iiab:masterfrom
deldesir:deldesir-description-1
Jan 31, 2024
Merged

Retrieve relevant description#116
holta merged 1 commit into
iiab:masterfrom
deldesir:deldesir-description-1

Conversation

@deldesir

Copy link
Copy Markdown
Member

🚀 Pull Request Overview:

This pull request retrieves the correct description for each video.

📋 Checklist:

  • Tested the changes thoroughly.

📌 Testing scenarios:
See Issue #97

cc @EMG70

PS This PR builds upon #110 to #114

@deldesir deldesir requested a review from holta January 31, 2024 13:14
@deldesir deldesir self-assigned this Jan 31, 2024
@holta holta mentioned this pull request Jan 31, 2024
2 tasks
@holta

holta commented Jan 31, 2024

Copy link
Copy Markdown
Member

FYI my understanding is this PR patches a small functionality regression introduced by:

@deldesir

Copy link
Copy Markdown
Member Author

Tested on LRN2:

Before (with incorrect description, prior to applying this PR):
before

After (with correct description, after applying this PR):
after

@EMG70

EMG70 commented Jan 31, 2024

Copy link
Copy Markdown

iiab-diagnostics http://sprunge.us/XPsd7T?en
Downloaded videos appear with full title in Tasks.The video url is now clickable.Good progress @deldesir ,your screenshots were very helpful in the testing .I will keep testing and update.
Screenshot from 2024-01-31 17-17-15

A scrollable full video description is showing Ok.see screenshot.
Screenshot from 2024-01-31 18-03-35

@EMG70

EMG70 commented Jan 31, 2024

Copy link
Copy Markdown

Just to add,the first video in the playlist shows progress until 62% and stalled ,the rest were in "waiting " status .Downloading progressed in the background until all were successfully downloaded.

@holta holta added the bug Something isn't working label Feb 14, 2024
@deldesir deldesir deleted the deldesir-description-1 branch July 1, 2024 21:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants