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

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

Open
siinghd opened this issue Aug 8, 2024 · 13 comments · May be fixed by #1337
Open

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

siinghd opened this issue Aug 8, 2024 · 13 comments · May be fixed by #1337
Labels
enhancement New feature or request high priority

Comments

@siinghd
Copy link
Collaborator

siinghd commented Aug 8, 2024

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 siinghd added enhancement New feature or request high priority labels Aug 8, 2024
@Sajal0208
Copy link

taking this up

@krishna9358
Copy link

@siinghd can work on it.

@Sajal0208
Copy link

Sajal0208 commented Aug 8, 2024

@krishna9358 should I work on it or not?

@siinghd have you started working on it?

@sand-storm-17
Copy link

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

@krishna9358
Copy link

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

@Sajal0208
Copy link

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

@sand-storm-17
Copy link

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

@iSouvikKhan
Copy link
Contributor

iSouvikKhan commented Aug 8, 2024

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

@krishna9358
Copy link

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

@siinghd
Copy link
Collaborator Author

siinghd commented Aug 8, 2024

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.

@iSouvikKhan
Copy link
Contributor

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

@shaurya35
Copy link
Contributor

@siinghd
still open for new PRs?

@ezhil56x ezhil56x linked a pull request Sep 27, 2024 that will close this issue
2 tasks
@shaurya35
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request high priority
Projects
None yet
6 participants