[codex] Add Revision History testimonial#94
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ 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 |
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
508-dynamic | 8e07c3e | Commit Preview URL Branch Preview URL |
Jul 02 2026, 09:36 AM |
There was a problem hiding this comment.
Pull request overview
This PR updates the portfolio page/components to support a new “Revision History” entry, including optional project links and testimonial quotes, while aiming to keep existing portfolio entries working with varying amounts of content.
Changes:
- Adds “Revision History” as the first portfolio item in the portfolio data JSON, including a URL and testimonial.
- Extends portfolio data plumbing (
url,testimonial, optionalimg) from the page into the portfolio item component. - Updates
PortfolioItemrendering to conditionally show a “Visit project” link and a testimonial block.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| src/pages/portfolio.astro | Extends the portfolio entry shape and passes url/testimonial into PortfolioItem. |
| src/data/portfolio.json | Adds the new “Revision History” portfolio item with link + testimonial. |
| src/components/PortfolioItem.astro | Conditionally renders optional project link and testimonial; makes several fields optional. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| interface PortfolioItemData { | ||
| subtitle: string; | ||
| title: string; | ||
| p: string[]; |
| content_p1: portfolioItem.p[0], | ||
| content_p2: portfolioItem.p[1], | ||
| content_p3: portfolioItem.p[2], | ||
| content_p4: portfolioItem.p[3], |
| href={url} | ||
| class="mt-4 inline-flex text-sm font-semibold leading-6 text-cyan-700 hover:text-cyan-600" | ||
| target="_blank" | ||
| rel="noreferrer" |
Summary
Validation
bun run astro checkpassed with 0 errors, 0 warnings, and 0 hints./portfolioserved Revision History before VoyTravel with the testimonial visible.Notes
bun run buildstill fails afterastro checkdue an existing dependency mismatch: Astro detects Vite 8 while this Astro version requires Vite 7 (Missing field tsconfigPaths...).