feat: Implement quiz API with enhanced functionality and UI #28
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…nto backend/member-quiz
Contributor
There was a problem hiding this comment.
Copilot reviewed 6 out of 17 changed files in this pull request and generated 3 comments.
Files not reviewed (11)
- .env.example: Language not supported
- package.json: Language not supported
- app/(user)/quiz/[slug]/QuizClient.tsx: Evaluated as low risk
- app/(user)/quiz/[slug]/page.tsx: Evaluated as low risk
- next.config.ts: Evaluated as low risk
- components/ArticleCard.tsx: Evaluated as low risk
- pnpm-lock.yaml: Evaluated as low risk
- lib/utils.ts: Evaluated as low risk
- app/(user)/quiz/page.tsx: Evaluated as low risk
- app/(user)/quiz/history/page.tsx: Evaluated as low risk
- app/(user)/quiz/[id]/QuizClient.tsx: Evaluated as low risk
Comments suppressed due to low confidence (3)
app/api/user/quizzes/history/route.ts:0
- The new API route for fetching quiz history should have corresponding tests to ensure its functionality is validated.
export const GET = async (req: NextRequest) => {
app/api/user/quizzes/[id]/route.ts:8
- The 'params' property should not be a Promise. It should be resolved before being used. Change the type to '{ id: string; }' and ensure it is resolved before usage.
params: Promise<{ id: string; }>;
app/api/user/quizzes/[id]/submit/route.ts:7
- The
paramsproperty should not be aPromise. It should be an object directly containing theid. Change it toparams: { id: string; };.
params: Promise<{ id: string; }>;
vzrenggamani
previously requested changes
Jan 9, 2025
Contributor
vzrenggamani
left a comment
vzrenggamani
left a comment
There was a problem hiding this comment.
Please seeks the commented file, especially in the secret key and functions modularism
Member
Author
Ahh okay, i completely forgot about the logging and the secret key. I will resolve the issues tomorrow morning |
DimzsArdiminda
approved these changes
Jan 9, 2025
DimzsArdiminda
approved these changes
Jan 9, 2025
DimzsArdiminda
requested review from
DimzsArdiminda and
vzrenggamani
and removed request for
vzrenggamani
…nto backend/member-quiz
…/gdsc-um/homesite-refactor into backend/member-quiz
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Introduce a quiz API, improve quiz functionality and UI components, and add features for result saving and quiz history. Refactor data structure and implement encryption for secure quiz retrieval.