Complete OrbitStack qualifying tasks - #8
Open
Phloraxx wants to merge 6 commits into
Open
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
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.
Submission checklist
Updated GitHub repository
https://github.com/Phloraxx/orbitstack/tree/fix/orbitstack-challenge
Working version
The application is working on the submitted branch and was verified with the test suite, TypeScript checks, Prisma validation, and a production Next.js build.
Bugs identified
discountCents.Fixes implemented
shippingCents = 0and is not credited again as a monetary discount.(userId, createdAt)index.POST /api/orders/[id]/duplicatewith authentication,404for missing orders,403for wrong-owner requests, current product/stock checks, and the existing{ data, error }response envelope.Remaining bugs / incomplete features
npm run lintstill reports 7 pre-existing errors and 4 warnings in unrelated UI/hydration/link code. These were present outside the challenge changes; changed-file lint passes and the production build succeeds.Run locally
Then open
http://localhost:3000.Verification
npm test— 9/9 tests passednpx tsc --noEmit— passednpx prisma validate— passednpm run build— passedgit diff --check— passedAI usage is documented in
AI_LOG.md.Screenshots/demo video were not included because they are optional in the challenge brief.