feat: add some description to the page#116
Conversation
- enrich the page summarizing the information for users to read it easily. - also helps with screen readers to digest this easier
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughThis PR enhances the company sponsor detail page with structured metadata and improved presentation. A new JSON-LD utility module generates schema.org schemas (Organization, BreadcrumbList, FAQ) for sponsorship data. A location formatting helper consolidates city/county display. The company route refactors to use these utilities for metadata head generation and narrative rendering. Header social links are rebranded from TanStack to NikilKuruvilla. ChangesCompany Sponsor Display with JSON-LD and Metadata
Header Branding Update
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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.
Actionable comments posted: 2
🤖 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.
Inline comments:
In `@apps/web/src/utils.ts`:
- Line 95: The location parts are being pushed using the raw value
(parts.push(titleCase(value))) which can contain surrounding whitespace while
the dedupe step is trim-aware; update the push to use a trimmed value (e.g.,
call value.trim() before titleCase) so you pass titleCase a trimmed string and
avoid extra spaces in rendered output—locate the parts.push/titleCase(value)
usage and apply trimming to the value variable.
In `@apps/web/src/utils/jsonld.ts`:
- Around line 47-48: The function currently always sets parts.addressCountry =
address.country || 'GB', causing a non-null PostalAddress when no real address
fields exist; change the logic so the default 'GB' is only applied if at least
one other address field is present (i.e., include addressCountry only when
address has other properties or when address.country is explicitly provided),
and keep returning null if the only contribution would be the synthetic default;
update the code around the parts object and the addressCountry assignment to
perform this conditional inclusion.
🪄 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: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: b01b2fd2-13c8-453f-9b16-c7bc49516a93
📒 Files selected for processing (4)
apps/web/src/components/Header.tsxapps/web/src/routes/company.$id.$slug.tsxapps/web/src/utils.tsapps/web/src/utils/jsonld.ts
Summary by CodeRabbit
New Features
Improvements