Conversation
Rebase PR KnightHacks#487 onto current main and place the LeetCode prompts in the live cron-owned message list.
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. 📝 WalkthroughWalkthroughThe Priority: ⬇️ Low Merge Risk: 🔵 Low · up to The dues link currently opens a general membership page rather than the payment flow. Point it to the dues page before relying on 🚥 Pre-merge checks | ✅ 8✅ Passed checks (8 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
There was a problem hiding this comment.
Note
Quiet mode is enabled, so only the most important comments were posted inline. Other review comments are grouped below.
🟡 Other comments (1)
apps/tk/src/consts/index.ts-53-53 (1)
53-53: 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick winLink directly to the dues page.
/linkslabelshttps://blade.knighthacks.org/as “pay your dues here,” but the root URL renders Blade’s general membership landing page. The dues flow is at/member/dues.Suggested fix
- "• [**Be a member and pay your dues here!**](https://blade.knighthacks.org/)", + "• [**Be a member and pay your dues here!**](https://blade.knighthacks.org/member/dues)",
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository: KnightHacks/forge/.coderabbit.yml
Review profile: QUIET
Plan: Advanced
Run ID: b711e2e0-755b-4f45-b5ba-08290d7151f6
📒 Files selected for processing (1)
apps/tk/src/consts/index.ts
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.
must108
left a comment
There was a problem hiding this comment.
why is the screenshot black and white? lol
also please update your branch there are some old commits in here
|
Wdym by black and white? You mean the color of the screenshot is not saturated? Also I already synced all possible commits to my branch. Idk if that's how the "sync fork" button works. |
must108
left a comment
There was a problem hiding this comment.
should be fine but please squash ur commits
also yeah doesnt look saturated or something
|
Gotcha. Also I lowered the UI saturation on purpose cuz I wanted a less distracting Discord experience. |
|
Replaced by #591, which is based on the latest upstream main and contains only the /links change. |
Why
The /links command from the TK bot currently contains links from previous Knight Hacks events and pages from 2024; for instance, Knight Hacks VII instead of the upcoming Knight Hacks IX.
What
Updated links for the /links command, including:
Screenshot
Test Plan
Ran the Discord bot on a burner server and tested the
/linkscommand. Ran all requiredpnpmcommands and passed all tests.Checklist
pnpm db:generateand committed the generated files inpackages/db/drizzle/