Partner rejection updates - #3671
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
📝 WalkthroughWalkthroughReduces the maximum character limit for partner rejection notes from 5000 to 500 and updates the corresponding UI components. Adds a character counter display in the rejection modal and refreshes tooltip styling and layout in the partner program card. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Poem
🚥 Pre-merge checks | ✅ 1 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Warning Review ran into problems🔥 ProblemsTimed out fetching pipeline failures after 30000ms 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.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
apps/web/ui/partners/program-card.tsx (1)
34-52:⚠️ Potential issue | 🔴 Critical
font-regis not a valid Tailwind utility and will be silently ignored.The class
font-regused on lines 45 and 93 is not defined in the Tailwind configuration. Standard Tailwind only providesfont-thinthroughfont-black. Since this utility doesn't exist, the font-weight styling will be lost. Check if this should befont-normal(400 weight) or if a customfont-regutility needs to be added to the Tailwind config.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@apps/web/ui/partners/program-card.tsx` around lines 34 - 52, The component uses a non-existent Tailwind utility "font-reg" (in the value paragraph class) so the font-weight is silently ignored; update the class on the element that renders {value} (and any other occurrences of "font-reg" in this file/component) to a valid utility like "font-normal" or change to a configured custom utility by adding "font-reg" to the Tailwind config; locate the class string inside the JSX (the valueClassName fallback and any hardcoded classnames around the {value} render) and replace "font-reg" with "font-normal" unless you intentionally add "font-reg" to your Tailwind theme.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Outside diff comments:
In `@apps/web/ui/partners/program-card.tsx`:
- Around line 34-52: The component uses a non-existent Tailwind utility
"font-reg" (in the value paragraph class) so the font-weight is silently
ignored; update the class on the element that renders {value} (and any other
occurrences of "font-reg" in this file/component) to a valid utility like
"font-normal" or change to a configured custom utility by adding "font-reg" to
the Tailwind config; locate the class string inside the JSX (the valueClassName
fallback and any hardcoded classnames around the {value} render) and replace
"font-reg" with "font-normal" unless you intentionally add "font-reg" to your
Tailwind theme.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: cb7d9441-9231-43f9-bf08-236e8bc711bc
📒 Files selected for processing (3)
apps/web/lib/zod/schemas/partners.tsapps/web/ui/modals/reject-partner-application-modal.tsxapps/web/ui/partners/program-card.tsx
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@apps/web/ui/partners/program-card.tsx`:
- Line 93: Replace the typo in the class string used for the ProgramCard
component: change the valueClassName prop value that currently contains
"font-reg" to "font-regular" so it matches the project's established utility
classname pattern (look for valueClassName="... font-reg ..." in
program-card.tsx and update to "font-regular").
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: f2587232-c4b7-4e3e-828f-25e260e84350
📒 Files selected for processing (1)
apps/web/ui/partners/program-card.tsx
Summary by CodeRabbit
Validation Changes
UI/UX Improvements