fix: ticket category on discord - #35
Conversation
Signed-off-by: Niv <148060940+Nivmizz7@users.noreply.github.com>
Thanks for using CodeAnt! 🎉We're free for open-source projects. if you're enjoying it, help us grow by sharing. Share on X · |
🏁 CodeAnt Quality Gate ResultsCommit: ✅ Overall Status: PASSEDQuality Gate Details
|
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughSummary by CodeRabbit
WalkthroughTicket creation now searches for the ChangesSupport category lookup
Estimated code review effort: 1 (Trivial) | ~2 minutes ✨ Finishing Touches📝 Generate docstrings
🧪 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 |
PR Code Suggestions ✨Latest suggestions up to commit
|
| Category | Suggestion | Severity | Generated at (UTC) |
| Incorrect condition logic |
Lowercasing only the channel name prevents the configured support category from ever matchingThe channel name is lowercased before comparison, but the expected literal still src/interactions/buttonHandlers.js [32] Why it matters? 🤔
Prompt for AI Agent 🤖This is a comment left during a code review.
**Path:** src/interactions/buttonHandlers.js
**Line:** 32:32
**Comment:**
*Incorrect Condition Logic: The channel name is lowercased before comparison, but the expected literal still contains uppercase `Support`. Therefore a category named `🆘 --- Support ---` becomes `🆘 --- support ---` and never matches, causing every ticket creation attempt to take the missing-category path. Lowercase the expected literal as well, or compare both names using the same normalization.
Validate the correctness of the flagged issue. If correct, How can I resolve this? If you propose a fix, implement it and please make it concise.
Once fix is implemented, also check other comments on the same PR, and ask user if the user wants to fix the rest of the comments as well. if said yes, then fetch all the comments validate the correctness and implement a minimal fix | Critical | 2026-08-14 18:30
|
Summary by cubic
Fixes ticket creation to target the Discord category named "🆘 --- Support ---" instead of "ticket", so new tickets are filed under the correct category.
Written for commit 077f477. Summary will update on new commits.