cms icon indicating copy to clipboard operation
cms copied to clipboard

feature: Connect the question to the video (High priority - valuable bounty)

Open siinghd opened this issue 1 year ago • 13 comments

We already have a platform for questions at 100xdevs, but it is currently underutilized.

To increase engagement, I propose connecting questions to specific videos. Here's a plan for implementation, which can be further improved for a better outcome:

Proposed Features:

  1. Linking Questions to Videos:

    • When creating a question, users should have the option to link it to a specific video.
    • Users can either search for a video or select one from a searchable dropdown menu.
    • This feature should be optional, as we still want to allow questions without associated videos.
  2. Video Integration:

    • Each video should have a button that, when clicked, redirects the user to all questions related to that video.
  3. Database Changes:

    • Add an optional video_id field to the question table to store the associated video ID.
  4. Future Improvements (V2):

    • Implement notifications to a specific Discord channel when a new question is created.

Note: These are initial thoughts, and any further improvements are welcome! (do not limit to only initial thoughts!)

siinghd avatar Aug 08 '24 17:08 siinghd

taking this up

Sajal0208 avatar Aug 08 '24 17:08 Sajal0208

@siinghd can work on it.

krishna9358 avatar Aug 08 '24 17:08 krishna9358

@krishna9358 should I work on it or not?

@siinghd have you started working on it?

Sajal0208 avatar Aug 08 '24 18:08 Sajal0208

@siinghd working on it, will create a PR. Do I need to provide video proof in pull request ?

sand-storm-17 avatar Aug 08 '24 21:08 sand-storm-17

@sand-storm-17 hii, I am already working on it. Thanks!

krishna9358 avatar Aug 08 '24 21:08 krishna9358

@krishna9358 @sand-storm-17 even I am working on it.

Sajal0208 avatar Aug 08 '24 21:08 Sajal0208

@krishna9358 @Sajal0208 okay you guys work on it. I'll start on it if there is no pull request by tomorrow.

sand-storm-17 avatar Aug 08 '24 22:08 sand-storm-17

image

getting this error upon committing can anyone tell me how can i resolve this ?? i did husky uninstall -> install no luck I am using windows

iSouvikKhan avatar Aug 08 '24 22:08 iSouvikKhan

@iSouvikKhan if it's coming while installing the packages, then install forcefully. npm install --force.... some packages are depricated !

krishna9358 avatar Aug 08 '24 22:08 krishna9358

The better PR will get merged only, make sure to not work quickly and push it just for the sake of time...i will review all and choose the better solution.

siinghd avatar Aug 08 '24 22:08 siinghd

Hi @krishna9358, resolved the issue using --no-verify flag while committing in terminal

iSouvikKhan avatar Aug 08 '24 22:08 iSouvikKhan

@siinghd still open for new PRs?

shaurya35 avatar Sep 18 '24 22:09 shaurya35

@devsargam please review #1340 Tried to implement most out of least code possible. Open for improvements

shaurya35 avatar Sep 28 '24 00:09 shaurya35

@devsargam i guess this can be closed too.

mrpaaradox avatar Oct 25 '24 11:10 mrpaaradox