Restore Mac App Store marketing, support, and privacy pages - #107
Conversation
Co-authored-by: offyotto <245704867+offyotto@users.noreply.github.com>
✅ All code review findings resolved.
Code Review ✅ Approved 1 closed / 1 findings🟡 Medium risk · Restoring three public App Store pages changes which website routes resolve. Restores three Mac App Store pages (
✅ 1 closed✅ Bug: Restored pages use CSS classes the current styles.css no longer has
Review coverage🤖 Auto-approval Approved 📋 Rules No rules evaluated 🧪 Functional validation Not enabled · Set up OptionsAuto-apply is off → Gitar will not commit updates to this branch. Comment with these commands to change the behavior for this request:
Was this helpful? React with 👍 / 👎 | Powered by Gitar — free for open source |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2e78c2f62c
ℹ️ 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".
Change
The App Store marketing, support, and privacy URLs return 404 after #103 deleted
docs/Mac-App-Store/.Restore these three pages from
0d6020a, the commit immediately before #103:docs/Mac-App-Store/index.htmldocs/Mac-App-Store/support/index.htmldocs/Mac-App-Store/privacy/index.htmlPreserve the page content and canonical URLs. Remove only the obsolete font preloads because #104 deleted that font.
GitHub Pages serves
main:/docsatxnu.is-a.dev. After merge and deployment, the existing GitHub URLs will redirect to the restored pages.The change leaves the README, homepage, and App Store Connect fields unchanged.
Verification
Confirmed that all three existing GitHub URLs redirect to
xnu.is-a.devand return HTTP 404.Served
docs/with a temporary local HTTP server. All three restored page routes returned HTTP 200 with the expected HTML.Passed 52 local link and asset checks, including fragment targets, styles, scripts, icons, and screenshots.
Compared each restored file with its historical source. Each file matches, except for the obsolete font preload.
Confirmed that the patch contains exactly three HTML files.
git diff --cached --checkpassed.Checks cover the changed behavior.
The restored pages document the App Store edition.
Security review: not applicable to this static HTML restoration.