Skip to content
This repository was archived by the owner on Feb 21, 2026. It is now read-only.

Blueprint Builder Page: Implement Form Integration#91

Open
Diego-Cano wants to merge 1 commit into
mainfrom
upgrade-90-blueprint-builder-page
Open

Blueprint Builder Page: Implement Form Integration#91
Diego-Cano wants to merge 1 commit into
mainfrom
upgrade-90-blueprint-builder-page

Conversation

@Diego-Cano
Copy link
Copy Markdown

Implemented the Blueprint Builder page and integrated a Google Form for blueprint submissions. Linked the page in the navigation bar and ensured consistent styling across the site.
week1

Copy link
Copy Markdown
Contributor

@el-riber el-riber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, I tried to test it, but it gives me the same error everytime
Screenshot 2024-11-13 at 12 21 58 AM

@bcko
Copy link
Copy Markdown
Contributor

bcko commented Feb 27, 2025

/gemini summary

@gemini-code-assist
Copy link
Copy Markdown

Summary of Changes

This pull request implements a new "Blueprint Builder" page for the Pelletier Construction Group website. The page includes a Google Form to allow users to submit their blueprint ideas. The PR also adds a link to the new page in the navigation bar of all existing pages, ensuring consistent styling across the site.

Highlights

  • New Feature: Implementation of the Blueprint Builder page with a Google Form integration for blueprint submissions.
  • Navigation Update: Addition of a 'Blueprint Builder' link to the navigation bar on all pages for easy access.
  • Code Quality: Adds a .hintrc file to disable the disown-opener hint.

Changelog

  • .hintrc
    • Added .hintrc file to disable the disown-opener hint.
  • about.html
    • Added a link to the 'Blueprint Builder' page in the navigation bar.
  • blueprint.html
    • Created a new 'Blueprint Builder' page with an embedded Google Form.
    • Included standard header and footer elements for consistency.
  • contact.html
    • Added a link to the 'Blueprint Builder' page in the navigation bar.
  • index.html
    • Added a link to the 'Blueprint Builder' page in the navigation bar.
  • package-lock.json
    • Added package-lock.json file
  • projects.html
    • Added a link to the 'Blueprint Builder' page in the navigation bar.
  • testimonials.html
    • Added a link to the 'Blueprint Builder' page in the navigation bar.

Activity

  • A request for a summary was made by @bcko via the /gemini command.

@bcko
Copy link
Copy Markdown
Contributor

bcko commented Feb 27, 2025

/gemini review

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

The pull request implements a new Blueprint Builder page with a Google Form integration. The page is linked in the navigation bar, and consistent styling is maintained across the site. Overall, the changes seem well-structured and functional. However, I have a few suggestions to enhance the code quality and maintainability.

Summary of Findings

Merge Readiness

The pull request introduces a new feature and integrates it well into the existing website structure. While the changes seem functional, I recommend addressing the suggestions provided to improve code quality and maintainability. I am unable to directly approve this pull request, and recommend that other users review and approve this code before merging. Given the absence of any CRITICAL or HIGH severity issues, the pull request can be merged after addressing the suggestions.

@Alzavio
Copy link
Copy Markdown

Alzavio commented Mar 10, 2025

Looks good, but the form has a separate scrollbar. If possible maybe want to get rid of that. Also the grey around the submit button kind of blends into the background. Any possibility that could be changed to white?

image

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

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

4 participants