Remove rebrand announcement from auth header - #615
Conversation
📝 WalkthroughWalkthrough
ChangesAuth header update
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Deploying maple with
|
| Latest commit: |
4260726
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://5fde7168.maple-ca8.pages.dev |
| Branch Preview URL: | https://codex-remove-maple-research.maple-ca8.pages.dev |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
frontend/src/components/AuthHeader.tsx (1)
45-45: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low valueLine exceeds 100-character limit.
The
MarketingHomeLinkclassName string on this line is well over the repository's 100-character line limit for TypeScript/React code. Consider extracting the class list into a constant or using template literals/cn()split across lines.As per coding guidelines, "Use 2-space indentation, double quotes, and enforce 100-character line limit for TypeScript/React code" for
frontend/src/**/*.{ts,tsx}.♻️ Suggested formatting
- <MarketingHomeLink className="col-start-2 row-start-1 inline-flex items-center gap-2 justify-self-end rounded-md px-3 py-2 text-sm font-semibold text-[`#747474`] transition hover:bg-black/5 hover:text-[`#221a18`] dark:text-muted-foreground dark:hover:bg-white/5 dark:hover:text-foreground" /> + <MarketingHomeLink + className={cn( + "col-start-2 row-start-1 inline-flex items-center gap-2 justify-self-end rounded-md px-3 py-2", + "text-sm font-semibold text-[`#747474`] transition hover:bg-black/5 hover:text-[`#221a18`]", + "dark:text-muted-foreground dark:hover:bg-white/5 dark:hover:text-foreground", + )} + />🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@frontend/src/components/AuthHeader.tsx` at line 45, Reformat the MarketingHomeLink usage in AuthHeader so no TypeScript/React line exceeds 100 characters. Split the className across lines using the existing styling approach, or extract the class list into a local constant while preserving all classes and behavior.Source: Coding guidelines
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@frontend/src/components/AuthHeader.tsx`:
- Line 45: Reformat the MarketingHomeLink usage in AuthHeader so no
TypeScript/React line exceeds 100 characters. Split the className across lines
using the existing styling approach, or extract the class list into a local
constant while preserving all classes and behavior.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: ac5b1b26-aa8d-4e90-bd9b-5ebd3ce4d8b2
📒 Files selected for processing (1)
frontend/src/components/AuthHeader.tsx
Summary
Validation
git diff --checkprettier: command not found)Summary by CodeRabbit