Skip to content

Remove legacy quote-form.js script includes from HTML pages#167

Open
GsCommand wants to merge 1 commit intomainfrom
codex/remove-legacy-quote-form.js-references
Open

Remove legacy quote-form.js script includes from HTML pages#167
GsCommand wants to merge 1 commit intomainfrom
codex/remove-legacy-quote-form.js-references

Conversation

@GsCommand
Copy link
Contributor

Motivation

  • The site moved to a newer quote form (Jobber embed) and the legacy /assets/js/quote-form.js was still being included site-wide, creating unnecessary requests and potential script conflicts.
  • Remove the unused includes with a minimal, low-risk change that does not touch the new form embed or form markup.

Description

  • Removed the <script src="/assets/js/quote-form.js" defer></script> include from 70 HTML pages (footer script blocks across about/, care-program/, commercial/, faq/, get-a-quote/, paver-sealing/, service-areas/, warranty/, etc.).
  • Deleted the leftover comment line referencing quote-form.js from /get-a-quote/index.html so the page only contains the active Jobber embed script.
  • Left the assets/js/quote-form.js file untouched in the repo and flagged it as unused so it can be removed in a follow-up cleanup commit.

Testing

  • Verified pre-change references with rg -n "quote-form\.js" --glob '*.html' and confirmed includes were found across many pages. (succeeded)
  • Ran a bulk removal script which updated 70 HTML files and verified no HTML files reference the legacy script with rg -n "quote-form\.js" || true. (succeeded — no matches)
  • Served the site locally and ran a Playwright check to confirm /get-a-quote/ still contains the Jobber embed script and representative pages no longer include the legacy script; the embed attempted to load but remote CORS prevented full iframe load (expected for local testing). (succeeded — embed tag present, legacy include absent)
  • Confirmed assets/js/quote-form.js still exists on disk but is no longer referenced by any HTML files and can be safely deleted in a later follow-up. (succeeded)

Codex Task

@vercel
Copy link

vercel bot commented Mar 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
hydrosealpavers-site Ready Ready Preview, Comment Mar 12, 2026 6:28pm

Request Review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant