feat: Add SYSmoAI marketing pages, WhatsApp CTA, font shim, and storefront resilience#1
feat: Add SYSmoAI marketing pages, WhatsApp CTA, font shim, and storefront resilience#1
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 631549dbe0
ℹ️ 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".
| @@ -0,0 +1,24 @@ | |||
| import { MetadataRoute } from 'next' | |||
|
|
|||
| export default function sitemap(): MetadataRoute.Sitemap { | |||
There was a problem hiding this comment.
Exempt sitemap.xml from country-code redirects
This root metadata route generates /sitemap.xml, but our middleware stack (proxy.ts matcher + handleStorefrontRoutes in features/storefront/middleware.ts) redirects any non-country-prefixed path to /{countryCode}/.... In practice, /sitemap.xml is redirected to something like /us/sitemap.xml, and there is no handler for that path, so crawlers get a 404 instead of the sitemap. Please either exclude sitemap.xml from middleware matching or generate the sitemap at a country-prefixed route that actually resolves.
Useful? React with 👍 / 👎.
| { path: '/privacy', priority: 0.4, freq: 'yearly' as const }, | ||
| { path: '/terms', priority: 0.4, freq: 'yearly' as const }, |
There was a problem hiding this comment.
Remove non-existent URLs from sitemap entries
The sitemap currently advertises /privacy and /terms, but there are no corresponding storefront page routes for these paths (the new pages added in this commit are /about, /services, /industries, /contact, and /blog). Emitting URLs that resolve to 404 reduces sitemap quality and wastes crawl budget; these entries should be removed or pointed at real routes.
Useful? React with 👍 / 👎.
chore: ignore nested node_modules directories
Motivation
Description
features/storefront/lib/data/regions.tsandfeatures/storefront/lib/data/collections.tssogetRegionreturnsnull,listRegionsreturns{ regions: [] }, and collections functions return{ collections: [], count: 0 }on error.regionandcollectionsare available (features/storefront/screens/HomePage.tsx).components/ui/whatsapp-float.tsxand mount it in the storefront main layout (features/storefront/screens/MainLayout.tsx).next/font/googleusage in storefront/dashboard logo and CTA components and add a webpack alias fornext/font/google/target.csswith a local shim filestyles/font-shim.css, plus system-font CSS variables and SYSmoAI brand tokens added toapp/globals.cssandnext.config.tsupdated.app/(storefront)/[countryCode]/(main)/(about,services,contact,blog,industries) and add a canonicalapp/sitemap.ts.Testing
npx next build --webpackwhich completed successfully and generated the site routes without font-fetch or GraphQL fatal errors.next/font/googleand other forbidden strings withrgand found no remaining matches in app/components/lib (scans returned no issues).Codex Task