Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Tip Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
npkhang99
left a comment
npkhang99
left a comment
There was a problem hiding this comment.
Overall, I think the core idea is solid.
Just a few minor fixes before merging.
Great work, btw.
| export async function POST(request: NextRequest) { | ||
| const { competitors, question } = (await request.json()) as { | ||
| competitors: Competitor[]; | ||
| question: string; | ||
| }; |
There was a problem hiding this comment.
Perhaps we need some kind of rate limiting for this route to make sure your demo app won't explode your API bill.
Also, capping the amount of competitors is not a bad idea here.
| typescript: { | ||
| ignoreBuildErrors: true, | ||
| }, |
There was a problem hiding this comment.
I’m not sure why this is here, but I would prefer it to be removed. Better to address the TypeScript errors properly
3 participants
Added the app competitor scout CLI app to the tinyfish-cookbook.