Add podcast SOC workflow field guide - #81
Conversation
Deploying hawkinsoperations-website with
|
| Latest commit: |
e212c7d
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://198951ea.hawkinsoperations-website.pages.dev |
| Branch Preview URL: | https://agent-podcast-readiness-rele.hawkinsoperations-website.pages.dev |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e212c7d0db
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| {teachingCards.map((card) => ( | ||
| <article className="podcast-card" key={card.src}> | ||
| <div className="podcast-card__visual"> | ||
| <img src={card.src} alt={card.alt} width="1920" height="1080" /> |
There was a problem hiding this comment.
All six podcast PNGs omit loading, so opening /podcast/ eagerly downloads approximately 2.9 MB of imagery even though most assets are several sections below the viewport. This unnecessarily delays and consumes bandwidth for mobile visitors; mark the below-fold images as lazy or use an image component that provides equivalent deferred loading.
Useful? React with 👍 / 👎.
| <a | ||
| className="cta cta-quiet podcast-header-link" | ||
| href="/podcast/" | ||
| aria-current={isCurrent("/podcast/") ? "page" : undefined} | ||
| > | ||
| Podcast guide | ||
| </a> |
There was a problem hiding this comment.
At widths just above 1024 px, the header's wrapping rule no longer applies while the primary navigation switches to the larger lg:gap-7 spacing; adding this always-visible podcast CTA makes the brand, seven navigation links, and CTA wider than the available container around 1025–1100 px. This can introduce horizontal overflow or squeeze/clip the brand at common landscape-tablet widths, so keep the CTA hidden until a wider breakpoint or extend the wrapped-header breakpoint.
Useful? React with 👍 / 👎.
1 participant
Objective
Publish a public-safe podcast field guide for From Logs to AI Triage: A Practical SOC Workflow for Beginners, integrate the six approved 16:9 visuals, and make the destination obvious from the HawkinsOperations homepage and responsive header.
Pages and components
/podcast/as the public presentation surface./podcast/to the sitemap.Podcast visual integration
public/podcast/.Validation
npm run check:site— PASSnpm run typecheck— PASSnpm run build— PASS; 48 static routes, including/podcast/npm run test:visual— PASS; 10/10 at 1920×1080, 1440×900, 768×1024, 430×932, and 390×844PODCAST_PREFLIGHT_READY=YESPODCAST_PREFLIGHT_READY=YESPODCAST_DEMO_READY=YESVisual QA covers zero horizontal overflow, six decoded 1920×1080 assets, responsive navigation visibility, 44 px mobile touch target, visible keyboard focus, changed internal route responses, the workflow anchor, and zero browser console errors.
Privacy and claim boundary
Cloudflare expectation
The existing Cloudflare Pages Git integration should build
npm run buildand publishdistfrom mergedmain. Deployment and the publichttps://hawkinsoperations.com/podcast/route must be verified after merge; merge alone is not treated as deployment proof.Rollback
Revert commit
e212c7d6onmainthrough a normal follow-up PR. No DNS, Cloudflare project settings, proof records, runtime state, or other repositories are changed by this PR.