Skip to content

Conversation

@addiebart
Copy link
Member

  • ICP Section for website
  • change permlink to /ipc26 to avoid repo conflict
  • fix judge applicaton link for ipc26

Copilot AI review requested due to automatic review settings January 28, 2026 17:16
@addiebart addiebart changed the title dev Fix link for judge application on /ipc26 Jan 28, 2026
@addiebart addiebart merged commit 8358ace into master Jan 28, 2026
4 checks passed
Copy link

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

Updates the IPC 2026 page to point the “Apply to be a Judge!” CTA at the correct judge application form.

Changes:

  • Replaced the judge application button link with a Google Forms URL on the IPC 2026 page.

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

<p class="text-2xl">And if you're interested...</p>
</div>
<Button {...({ text: 'Apply to be a Judge!', href: 'https://discord.gg/SgFkFv4bUR', target: '_blank', variant: 'primary'})} />
<Button {...({ text: 'Apply to be a Judge!', href: 'https://docs.google.com/forms/d/e/1FAIpQLSc5nc9ao4G5IkxVT9XgSKuLj11Vcw1GOUyp21IQ8BzwhOr60Q/viewform', target: '_blank', variant: 'primary'})} />
Copy link

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

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

This Button usage wraps an inline object in a spread ({...({ ... })}), which is inconsistent with other Button usages in the codebase and makes the markup harder to read. Prefer passing props as attributes (e.g., variant, href, target, text/children) or extracting the call-to-action object to a named constant if you need to spread it.

Suggested change
<Button {...({ text: 'Apply to be a Judge!', href: 'https://docs.google.com/forms/d/e/1FAIpQLSc5nc9ao4G5IkxVT9XgSKuLj11Vcw1GOUyp21IQ8BzwhOr60Q/viewform', target: '_blank', variant: 'primary'})} />
<Button
text="Apply to be a Judge!"
href="https://docs.google.com/forms/d/e/1FAIpQLSc5nc9ao4G5IkxVT9XgSKuLj11Vcw1GOUyp21IQ8BzwhOr60Q/viewform"
target="_blank"
variant="primary"
/>

Copilot uses AI. Check for mistakes.
<p class="text-2xl">And if you're interested...</p>
</div>
<Button {...({ text: 'Apply to be a Judge!', href: 'https://discord.gg/SgFkFv4bUR', target: '_blank', variant: 'primary'})} />
<Button {...({ text: 'Apply to be a Judge!', href: 'https://docs.google.com/forms/d/e/1FAIpQLSc5nc9ao4G5IkxVT9XgSKuLj11Vcw1GOUyp21IQ8BzwhOr60Q/viewform', target: '_blank', variant: 'primary'})} />
Copy link

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

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

PR description mentions changing the permalink to /ipc26, but this PR’s diff only updates the judge application href on this page. If a route/permalink change is still required, it’s not reflected in the current changeset (or may already have been applied elsewhere).

Copilot uses AI. Check for mistakes.
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.

2 participants