Skip to content

feat: Add QuestBriefSchema fields to quest creation API (#257) - #447

Open
Adil2009700 wants to merge 6 commits into
mainfrom
fix/questbrief-schema-fields-257
Open

feat: Add QuestBriefSchema fields to quest creation API (#257)#447
Adil2009700 wants to merge 6 commits into
mainfrom
fix/questbrief-schema-fields-257

Conversation

@Adil2009700

Copy link
Copy Markdown
Collaborator

Fixes #257 ## Problem QuestBriefSchema structured fields were defined in Prisma schema but not used by API. ## Solution Updated quest creation/update API to accept all QuestBriefSchema fields: - submissionInstructions, expectedDeliverables, attachments, clientResources - problemStatement, starterRepo, estimatedHours, squadSize, ipTerms, clientOrg ## Impact - Companies can provide structured quest briefs via API - Enables AI PM layer to parse/decompose quests - Students get clearer context and deliverables ## Next Steps 1. Update frontend forms to include structured fields 2. Update quest detail pages to display structured fields 3. Update TypeScript types to include new fields

Adil2009700 added 6 commits July 5, 2026 23:24
…#410) - Add questSubmissions include to getAssignments API to fetch latest submission data - Create ReworkFeedbackPanel component to display criteria results, quality score, and reviewer notes - Show feedback panel when assignment status is 'needs_rework' - Handle graceful fallback when no feedback data is available
… Wrap handleApprove and handleReject in try/catch blocks - Check HTTP response status and JSON success flag - Display error toast on failure, success toast on success - Use finally block to ensure loading state is reset
@Adil2009700
Adil2009700 requested a review from LarytheLord as a code owner July 23, 2026 16:48
@vercel

vercel Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
adventurers-guild Error Error Jul 23, 2026 4:48pm

@LarytheLord

Copy link
Copy Markdown
Owner

The QuestBriefSchema fields in app/api/company/quests/route.ts are real, correct, and still needed for #257 — nice work. Same note as #445/#446 though: this branch carries the same stale/duplicate changes (qa-queue page, email-change routes, xp-utils, assignment-service) from being stacked on top of those branches. Could you rebase this (and the other two) onto current main so each PR only contains its own actual scope? Makes review and merge much faster on all three.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CRITICAL: QuestBriefSchema fields missing from Prisma schema — quest context silently broken

2 participants