-
-
Notifications
You must be signed in to change notification settings - Fork 13
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
Better UI : videos pages #54
Comments
Would like to work on this. |
@benoit74 As per my understanding, for non topic content, each slug is basically a html generated by Jinja.
Please guide me whether it will be an optimal approach. |
@ayushanand308 this is probably not the easier issue to solve, please confirm you've at least achieved to start the scraper in @githyuvi yes, you understood the idea. It is the same idea for all kind of nodes (videos here, but also exercises, HTML5 content, audio, HTML document, and ePub and PDF You will probably need more than url to the video in the JSON file (e.g. video title, ...), I don't recall the exact final information you will need. You could either have one JSON per video node as you said (but it means many very small files, but we probably do not care) or we could bundle all of them in a single file (might be too big for nothing when you only want to play one video). To be analyzed / discussed. @ayushanand308 @githyuvi please synchronize on who will work on which issue, as you see there is many different kind of nodes where the same kind of work is needed, you could definitely split the work. You could probably discuss this together on Kiwix Slack if you want (do not discuss it here please, this issue is not a live chat). @ayushanand308 could even benefit from the good understanding @githyuvi already got on this scraper. |
This is a subtask of #42
Provide an enhanced UI of videos pages, based on Endless UI (e.g https://key.endlessos.org/en/explore/#/topics/c9d7f950ab6b5a1199e3d6c10d7f0103).
videos assets should be placed in a
./videos
subfolder in the ZIMThe text was updated successfully, but these errors were encountered: