Update engagement pricing and founder bio - #190
Conversation
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request updates the site's engagement offerings by removing explicit pricing and streamlining the user journey. By replacing specific price points with a direct link to the contact page, the site encourages more personalized communication with potential clients. The changes include updates to the UI, data structures, and test suite to ensure consistency and prevent future regressions. Highlights
New Features🧠 You can now enable Memory (public preview) to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize the Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counterproductive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (2)
WalkthroughThe Work With Us engagement cards no longer display prices and now use “More info” links to ChangesEngagement CTA update
Founder metadata update
Estimated code review effort: 3 (Moderate) | ~20 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Comment |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request removes pricing information from the "Work with Us" page engagements, updating the Vue component, TypeScript interfaces, fixture data, and unit tests accordingly. It also changes the call-to-action link text to "More info" and updates its styling. The reviewer noted an accessibility (a11y) issue where using repetitive "More info" links without context can hinder screen readers, and suggested adding an aria-label to provide descriptive context for each engagement option.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 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 `@tests/pages/WorkWithUsPage.test.ts`:
- Line 34: Broaden the currency-removal assertion in the WorkWithUsPage test so
it rejects all site-supported displayed currency formats, including spaced
dollar amounts, decimal/k suffixes, pound and euro values, and trailing USD
amounts. Keep the assertion focused on rendered text and ensure legacy price
values cannot pass unnoticed.
🪄 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: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Run ID: 46a77c14-b443-49df-bbac-7280e8f77658
📒 Files selected for processing (4)
src/pages/WorkWithUsPage.vuesrc/support/content-types.tsstorage/fixtures/work-with-us-page.jsontests/pages/WorkWithUsPage.test.ts
💤 Files with no reviewable changes (1)
- src/support/content-types.ts
gocanto
changed the title
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 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 `@tests/support/seo-content.test.ts`:
- Around line 50-58: Update the founder metadata test around the “connects the
organization founder metadata” case to parse the organization JSON-LD from
indexHtml and compare its complete founder object with
ORGANIZATION_JSON_LD.founder, covering `@type`, name, and url instead of checking
only the URL substring.
🪄 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: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Run ID: 02df5193-b61f-4ae5-a897-1f23e14c85c7
📒 Files selected for processing (8)
index.htmlsrc/pages/AboutPage.vuesrc/support/content-types.tssrc/support/seo.tsstorage/fixtures/about-page.jsonstorage/fixtures/site.jsontests/pages/AboutPage.test.tstests/support/seo-content.test.ts
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 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 `@src/pages/WorkWithUsPage.vue`:
- Line 54: Update the RouterLink in WorkWithUsPage to derive its aria-label from
engagement.cta.label, matching the link’s displayed text instead of hard-coding
“More info.”
🪄 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: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Run ID: ce96cd32-13e3-4b05-85f3-d1783b85c403
📒 Files selected for processing (3)
src/pages/WorkWithUsPage.vuetests/pages/WorkWithUsPage.test.tstests/support/seo-content.test.ts
1 participant
Summary
https://gocanto.shrel="me"identity metadata and founderPersonstructured dataWhy
Engagement pricing should no longer be published on the site, so visitors are directed to the contact page for more information. The founder bio and SEO metadata now use Gustavo Ocanto's canonical website.
Validation
pnpm exec vitest run tests/pages/WorkWithUsPage.test.tspnpm exec vitest run tests/pages/AboutPage.test.ts tests/support/seo-content.test.tspnpm run lint(passes with three existing unrelated warnings)pnpm run build/contactrel="me", and JSON-LD URLSummary by CodeRabbit