Repository navigation
Refresh About page copy and add tour / where-to-buy CTAs - #133
Merged
Merged
Conversation
Replace the body copy with the updated customer-facing text (no em dashes), rename "The Island" section to "Brewing on Saba", and add two CTA pairs (intro + closing): a product-agnostic TourInquiryCta with cta_location="about_page" and a Button-as-Link to /where-to-buy using the existing where_to_buy_click analytics attributes. Two MDX-specific adjustments keep the CTAs rendering correctly inside prose-dd: CTA labels stay inline so MDX does not paragraph-wrap them (a <p> inside the button inherited dark prose color on the dark button), and the unlayered .prose-dd anchor rules are scoped to classless links so they no longer override Button-as-Link styling. Closes #131 Generated with [Devin](https://devin.ai) Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Reviewer's GuideRefreshes the About page voice and structure, adds duplicated tour and where-to-buy CTA groups using existing conversion architecture, scopes prose link styling so component buttons render correctly, and introduces regression coverage for content, accessibility behavior, handoffs, and analytics. Sequence diagram for About page tour inquiry CTAsequenceDiagram
actor Visitor
participant AboutPage
participant TourInquiryCta
participant InquiryDialog
participant WhatsApp
Visitor->>AboutPage: Click Book a Brewery Tour
AboutPage->>TourInquiryCta: Open CTA
TourInquiryCta->>InquiryDialog: Show product-agnostic dialog
InquiryDialog-->>Visitor: Display Brewery Tour and Tour + Tasting choices
Visitor->>InquiryDialog: Select inquiry option
InquiryDialog->>WhatsApp: Handoff with cta_location about_page
File-Level Changes
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Rewrites the
/aboutbody copy in the updated customer-facing voice and adds two CTA pairs so visitors can act on the page: Book a Brewery Tour (opens the existing product-agnosticTourInquiryCtadialog with the Brewery Tour / Tour + Tasting choice) and Find Deep Dive Near You / Find Where to Buy (internalLinkto/where-to-buy).Closes #131
Changes
app/(pages)/about/page.mdx— new body copy per issue;## The Island→## Brewing on Saba; CTA groups after the intro and at the end. Metadata unchanged (still accurate).app/globals.css—.prose-dd a→.prose-dd a:not([class]). These rules are unlayered, so they were overriding the button utility classes on the Link CTAs (blue underlined text inside a button). Classless markdown anchors keep identical styling.smoke-tests/smoke.spec.ts— new test: refreshed copy present,Brewing on Sabaheading, oldThe Island/IGY Marinacopy gone, no em dashes in the article, both CTA pairs render with/where-to-buyhrefs, nowa.mebypass, and no<p>inside the tour buttons (regression guard — MDX paragraph-wraps multiline JSX children, which hid the label via the prose color rule).smoke-tests/tour-inquiry.spec.ts— new tests: both about-page CTAs open the product-agnostic dialog (2 radios, no event on open, Escape returns focus) and the handoff attributescta_location: "about_page"with the correct WhatsApp message.smoke-tests/analytics.spec.ts— new test: the About Where-to-Buy link fireswhere_to_buy_clickonce withcta_location: "about_page".No content in
TourInquiryCta, WhatsApp message generation, or analytics plumbing was touched — the About CTAs reuse the same architecture as the homepage hero.Verification
npx tsc --noEmit— passnpm run lint— passnpm test— 355/355npm run build— passnpx playwright test— 77/77 (smoke, tour-inquiry, analytics, accessibility, seo)npm run check:md-links— passnpm audit --omit=dev— 0 vulnerabilitiesRisk / deployment notes
None — content and presentation only. The
.prose-ddselector change affects only anchors that carry a class attribute; markdown body links (none currently exist in MDX pages) are unaffected.Generated with Devin
Summary by Sourcery
Refresh the About page content and provide visitors with accessible tour-booking and beer-finding actions.
New Features:
Bug Fixes:
Enhancements:
Tests: