Updating marketing site with seasonal influences - #211
Merged
Merged
Conversation
Co-authored-by: KevinTCoughlin <706967+KevinTCoughlin@users.noreply.github.com>
Agent-Logs-Url: https://github.com/cascadiacollections/shoutkit/sessions/0e6c7e7e-9782-4c4b-ad5f-001b8e743116 Co-authored-by: KevinTCoughlin <706967+KevinTCoughlin@users.noreply.github.com>
Copilot created this pull request from a session on behalf of
KevinTCoughlin
September 13, 2026 21:36
View session
KevinTCoughlin
marked this pull request as ready for review
September 13, 2026 21:36
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
There was a problem hiding this comment.
🟡 Changes recommended
Unresolved moderate issues remain in the homepage metadata/copy and published social-card asset/copy.
Get a fresh assessment by requesting another Copilot review.
Pull request overview
Refreshes the marketing site with a September coastal theme and updated TestFlight/Mac Catalyst messaging.
Changes:
- Updates homepage styling, metadata, availability copy, platform details, and FAQ.
- Revises social-card artwork and messaging.
- Documents the seasonal marketing rationale in
DECISIONS.md.
File summaries
| File | Summary | Final review comments |
|---|---|---|
site/og-image.html |
Updates social-card design and copy. | Moderate (3 votes): Regenerate and commit the published og-image.png. Moderate (1 vote): Scope TestFlight beta wording to iPhone/iPad; Mac is a source preview. |
site/index.html |
Updates site styling, metadata, and platform messaging. | Moderate (1 vote): Separate iPhone/iPad beta and Mac source-preview wording in image alt text. Moderate (1 vote): Update the homepage lastmod in site/sitemap.xml. |
DECISIONS.md |
Records the seasonal marketing rationale and availability wording. | No final comments. |
Review details
Suppressed comments (3)
site/index.html:24
- The image alt text also groups Mac with “now in TestFlight beta,” unlike the page's precise availability language. Screen readers and platforms that expose this metadata can therefore announce a Mac beta that is not distributed; describe the iPhone/iPad beta and Mac source preview separately.
<meta property="og:image:alt" content="Holmdel — internet radio for iPhone, iPad and Mac, now in TestFlight beta." />
site/index.html:7
- This is a substantive homepage/SEO update, but
site/sitemap.xmlstill reports the homepagelastmodas 2026-08-12. That makes the sitemap freshness metadata inaccurate and can delay recrawling the new beta/seasonal copy; update the homepage entry with this change.
<title>Holmdel — free, open-source internet radio for iPhone, iPad & Mac</title>
<meta name="description" content="Holmdel is a native SwiftUI internet radio app, now in TestFlight beta for iPhone and iPad with a Mac Catalyst preview available from source. Live stations, track metadata, Siri, CarPlay, Apple Watch and Apple TV — no ads, accounts or tracking." />
site/og-image.html:104
- This social-card sentence makes “Mac” part of “Now in TestFlight beta,” but the availability copy and decision record define TestFlight as iPhone/iPad-only and Mac Catalyst as a source preview. Scope the beta wording to iPhone/iPad here so the card does not advertise a Mac TestFlight build.
<p>Internet radio for iPhone, iPad & Mac. Now in TestFlight beta.</p>
- Files reviewed: 3/4 changed files
- Comments generated: 1
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Comment on lines
+103
to
+104
| <h1>One more song before the light goes.</h1> | ||
| <p>Internet radio for iPhone, iPad & Mac. Now in TestFlight beta.</p> |
Contributor
Author
There was a problem hiding this comment.
Regenerated and committed site/og-image.png from the updated source in 523c31e. I also rerendered it locally at 1200×630 and confirmed the committed PNG is byte-identical.
Contributor
|
@copilot Fix the code for all comments in this review thread. When a review comment includes a suggested change, apply the suggestion exactly. Do not make changes beyond what is described in the linked review thread. |
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.
Pull request created by AI Agent