Collapse forms to one mailbox, remove brand assets page - #22
Merged
Conversation
Every form on the site now submits with NEXT_PUBLIC_WEB3FORMS_KEY and delivers to contact@invisionsolutions.co.uk, matching the Web3Forms account, which is now a single form. /enquiries was seven department forms reading seven per-mailbox keys. It is now one general enquiry form. /contact and the testimonial form on /testimonials point at the same key. Nothing reads the NEXT_PUBLIC_WEB3FORMS_KEY_* variables any more. That is deliberate: those forms have been deleted from the account, so a stale variable left set in the hosting dashboard is now inert rather than live. Verified by building with distinct dummy values for the main key and three stale ones, and confirming only the main key reaches the bundle. The stale-key path is the exact fault that stopped /contact delivering in August. /contact's cross-link promised routing to a specific desk, which is no longer true, so it is replaced with the public address. Brand assets removed: the /brand route, the 15 generated files in public/brand/, and scripts/generate-brand-assets.mjs, along with the brand script, its CI reproduction guard, and the fontkit and wawoff2 dependencies. src/lib/brand.ts stays, since the logo, favicon and Open Graph card are drawn from it. The direct sharp pin goes with the generator. sharp still resolves to 0.35.4 as an optional dependency of next, above the advisory range, and npm audit reports zero. Its install-script allowlist entry is removed as stale: 0.35.x ships no install script, so there has been nothing to permit since the 0.34.5 bump, and an unused permit would let a reintroduced script through unreviewed. Removed the "Session: closed" line from the footer.
/services published sales@ and /testimonials published hello@. Both mailboxes are being retired alongside the per-department forms, so advertising them would leave dead addresses on a live site. A mailto never touched the Web3Forms account, so this is about where mail can actually be read rather than the linked-email allowance. contact@invisionsolutions.co.uk is now the only address the site publishes anywhere: 57 occurrences across the rendered pages, and no occurrences of sales@, hello@ or info@.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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
2 participants
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.
Matches the site to the Web3Forms account, which is now a single form delivering to
contact@invisionsolutions.co.uk.Forms
/enquirieswas seven department forms reading seven per-mailbox keys. It is now one general enquiry form./contactand the testimonial form on/testimonialssubmit with the same key.Nothing reads the
NEXT_PUBLIC_WEB3FORMS_KEY_*variables any more, deliberately. Those forms have been deleted from the account, so a stale variable left set in Vercel is now inert rather than live. That path is the exact fault that stopped/contactdelivering in August.The testimonial form was reading
NEXT_PUBLIC_WEB3FORMS_KEY_HELLO, which names a deleted form. Left alone it would have failed silently on the next deploy./contactcarried a cross-link promising routing to a specific desk. That is no longer true, so it is replaced with the public address.Published addresses
/servicespublishedsales@and/testimonialspublishedhello@. Both mailboxes are being retired, so advertising them would leave dead addresses on a live site. A mailto never touched the Web3Forms account, so this is about where mail can be read, not the linked-email allowance.contact@invisionsolutions.co.ukis now the only address the site publishes anywhere.Brand assets
Removed the
/brandroute, the 15 generated files inpublic/brand/,scripts/generate-brand-assets.mjs, thebrandscript, its CI reproduction guard, and thefontkitandwawoff2dependencies.src/lib/brand.tsstays. The logo, favicon and Open Graph card are all drawn from it.The direct
sharppin goes with the generator that used it.sharpstill resolves to 0.35.4 as an optional dependency of Next, above the advisory range, andnpm auditreports zero. Its install-script allowlist entry is removed as stale: 0.35.x ships no install script, so it has permitted nothing since the 0.34.5 bump in #19, and an unused permit would let a reintroduced script through unreviewed.Footer
Removed the
Session: closed — log {year} — access: grantedline.Verification
lint,tsc --noEmit,check:install-scriptsandbuildall clean/brandgone/enquiriesrenders one form and one textarea, with no department labelsSession: closedabsent from every rendered page; no/brandlinks renderedcontact@invisionsolutions.co.ukacross the rendered pages, and none ofsales@,hello@orinfo@Positive control on the inertness claim. Built with the main key and three stale keys set to distinct dummy UUIDs. The main key appears 13 times in the bundle;
_CONTACT,_HELLOand_SALESappear zero times each. The positive arm makes the zeros meaningful.Needed in Vercel after merge
Delete the seven
NEXT_PUBLIC_WEB3FORMS_KEY_SALES/_CONTACT/_SUPPORT/_BILLING/_INVOICE/_ADMIN/_HELLOvariables, confirmNEXT_PUBLIC_WEB3FORMS_KEYholds the Main Contact Form key, and redeploy. These values are inlined at build time, so a variable change alone has no effect on an existing deployment.