-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
feat: introducing Qs&As, reddit like tiny platform #436
Conversation
Comments/role
pull main, build fix
are you thinking off adding a subreddit sort of thing as well, if yes then till which depth. @siinghd. |
This is just like that, looks same as that |
@hkirat veere can you look at this? |
wow nice |
@hkirat this is global thing, i think for video we could just expand the comments. We already have different type of comments. In this somehow we could reference the video... like github does with pr's |
Can we put this behind a globally configurable feature flag @siinghd ? |
Maybe generally add a frontend feature flag functionality if possible |
@hkirat sure let me do it |
@hkirat should be done, maybe its dumb but i have added env var NEXT_PUBLIC_DISABLE_FEATURES = "featurea,featureb,featurec" if you add Maybe u also want to disable some things on clientside (like conditional rendering) thats why the env is with NEXT_PUBLIC |
MMMM build has some problems with const session = await getServerSession(authOptions), let me check |
/bounty $600 |
@@ -13,12 +13,18 @@ const voteHandler = async ( | |||
data: InputTypeHandleVote, | |||
): Promise<ReturnTypeHandleVote> => { | |||
const session = await getServerSession(authOptions); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ideally should take weather it is a vote or downvote as an input
Toggling can lead to inconsistencies if the user is in two tabs/if the tab hasn't updated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good follow up to add @siinghd
// appxCourseId: number; | ||
// openToEveryone: boolean; | ||
// slug: string; | ||
// discordRoleId: string; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we get rid of it?
PR feat:
Introducing tiny reddit like platform where people can ask questions or post something and people can reply.
In the prisma Schema two models are seperated (questions,answers) because with single recursive model it would be hard to get recursive answers.
Resolves #[Issue Number if there]
Checklist before requesting a review