Skip to content

Wire landing page REGISTER NOW CTA to external Google Form#31

Merged
HenriqueSFernandes merged 2 commits into
redesign/ctf-2026from
copilot/update-register-button-redirection
May 7, 2026
Merged

Wire landing page REGISTER NOW CTA to external Google Form#31
HenriqueSFernandes merged 2 commits into
redesign/ctf-2026from
copilot/update-register-button-redirection

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 7, 2026

Updates the landing page CTA so REGISTER NOW actually routes participants to the registration form, while preserving the existing button styling/component usage and client-component behavior.

  • CTA behavior update

    • Replaced the no-op click handler in src/app/page.tsx with an external navigation handler to:
      • https://forms.gle/GsPPQqYJ9RdNvbLCA
    • Opens in a new tab with safe window features (noopener,noreferrer).
  • Minimal file-local cleanup

    • Kept the change scoped to src/app/page.tsx.
    • Applied a small import ordering adjustment in the same file to keep it clean and consistent.
<Button
  size="lg"
  className="animate-gold-pulse"
  onClick={() =>
    window.open(
      "https://forms.gle/GsPPQqYJ9RdNvbLCA",
      "_blank",
      "noopener,noreferrer",
    )
  }
>
  REGISTER NOW
</Button>

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • fonts.googleapis.com
    • Triggering command: /usr/local/bin/node /usr/local/bin/node /home/REDACTED/work/CTF-Website/CTF-Website/node_modules/next/dist/compiled/jest-worker/processChild.js (dns block)
    • Triggering command: /usr/local/bin/node /usr/local/bin/node /home/REDACTED/work/CTF-Website/CTF-Website/node_modules/next/dist/server/lib/start-server.js (dns block)
  • workers.cloudflare.com
    • Triggering command: /usr/local/bin/node node /home/REDACTED/work/CTF-Website/CTF-Website/node_modules/.bin/next build (dns block)
    • Triggering command: /usr/local/bin/node /usr/local/bin/node /home/REDACTED/work/CTF-Website/CTF-Website/node_modules/next/dist/compiled/jest-worker/processChild.js (dns block)
    • Triggering command: /usr/local/bin/node node /home/REDACTED/work/CTF-Website/CTF-Website/node_modules/.bin/next dev --turbopack --hostname 0.0.0.0 --port 3000 (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

Create a PR that updates the landing page register button to redirect to the Google Forms registration link.

Repo: acmfeup/CTF-Website
Base branch: redesign/ctf-2026

Context:

  • File to change: src/app/page.tsx
  • Current REGISTER NOW button uses an empty onClick handler.

Requirements:

  1. Make the REGISTER NOW CTA open https://forms.gle/GsPPQqYJ9RdNvbLCA.
  2. Prefer opening in a new tab safely (noopener,noreferrer) since it’s an external URL.
  3. Keep styling and existing Button component usage intact.
  4. Ensure the code remains a valid Next.js client component.

Acceptance criteria:

  • Clicking REGISTER NOW navigates to the provided URL.
  • No lint/type errors introduced.

Reference URL:

The following is the prior conversation context from the user's chat exploration (may be truncated):

User: I want the regjster button to redirect to https://forms.gle/GsPPQqYJ9RdNvbLCA

Assistant: Update the REGISTER NOW button to navigate to the external Google Forms URL on click (in a new tab).

Change this part:

<Button size="lg" className="animate-gold-pulse" onClick={() => {}}>
	REGISTER NOW
</Button>

to:

<Button
	size="lg"
	className="animate-gold-pulse"
	onClick={() => window.open("https://forms.gle/GsPPQqYJ9RdNvbLCA", "_blank", "noopener,noreferrer")}
>
	REGISTER NOW
</Button>

If you want it to open in the same tab instead, use:

onClick={() => (window.location.href = "https://forms.gle/GsPPQqYJ9RdNvbLCA")}

If you want, I can commit this change to acmfeup/CTF-Website on the redesign branch—confirm the branch name you want to target.

User: Yes commit to redesign/ctf-2026 then open a pr

Assistant: I’ve kicked off the change and PR creation flow; I’m now waiting for your confirmation in the GitHub prompt to proceed with committing to redesign/ctf-2026 and opening the pull request.

This pull request was created from Copilot chat.

Agent-Logs-Url: https://github.com/acmfeup/CTF-Website/sessions/d600ae6d-8a4e-4b04-97bc-7ca891526ec3

Co-authored-by: HenriqueSFernandes <85371550+HenriqueSFernandes@users.noreply.github.com>
Copilot AI changed the title [WIP] Update landing page register button to redirect to Google Forms Wire landing page REGISTER NOW CTA to external Google Form May 7, 2026
Copilot AI requested a review from HenriqueSFernandes May 7, 2026 19:06
@HenriqueSFernandes HenriqueSFernandes marked this pull request as ready for review May 7, 2026 19:08
Copilot AI review requested due to automatic review settings May 7, 2026 19:08
@HenriqueSFernandes HenriqueSFernandes merged commit 41147cb into redesign/ctf-2026 May 7, 2026
2 of 3 checks passed
@HenriqueSFernandes HenriqueSFernandes deleted the copilot/update-register-button-redirection branch May 7, 2026 19:08
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Wires the landing page’s primary CTA so participants can actually reach the external registration form, keeping the existing Button component and client-component behavior intact.

Changes:

  • Replaced the previous no-op onClick handler on the REGISTER NOW button with a window.open call to the Google Form.
  • Opens the registration link in a new tab using noopener,noreferrer.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants