labrocadabro / communitytaught

MIT License
81 stars 31 forks source link

feature request: video tags / grouping #27

Open ht-l1 opened 1 year ago

ht-l1 commented 1 year ago

Hey can video tags added to support users in filtering which types of videos they like to see? for example when i search code related, all the html css js comes up. and when i search career, i will see networkings, crafting your stories etc.

labrocadabro commented 10 months ago

This is an excellent idea!

nmpereira commented 10 months ago

Would love to take this!

nmpereira commented 10 months ago

Some proof of concept, let me know if you have any thoughts! (pardon the bad styling, that's a WIP)

Lesson card:

image

Lesson page: image

Edit lesson: image

nmpereira commented 10 months ago

Was thinking of just adding the tags and no search bar. Since a user can just use control+f, for now, it's a very usable solution. A search bar can be an additional PR/issue.

Was thinking of adding tags to homework items as well but ill add some concepts here when I do em

labrocadabro commented 10 months ago

This looks great!

Search is another on my (long!) list of possible improvements. I agree it should probably be a separate issue, especially because what I'd like to do is make the transcripts of each video searchable.

From the screenshots you've got here it looks like tags are just text? Usually they'd be links you can click, which will take you to a new page showing all videos containing that tag.

nmpereira commented 10 months ago

I didn't realize there was support for tag links, I can also add that!

+1 on searchable transcripts. This might be something I also want to do, sounds fun to work on! I'll create an issue and put my thoughts on the plan of attack for it!

labrocadabro commented 10 months ago

Haha I'm not sure I would call it support , we'd have to add the code to make it work. We'd need to add the routes and the controller/db query to get the appropriate videos. If you're adding the tags as an array (not too hard to do), then it shouldn't be too difficult to implement, though.