Skip to content

Chopped ctf - #484

Merged
J164 merged 23 commits into
mainfrom
CTF
Feb 28, 2026
Merged

J164 merged 23 commits into
mainfrom
CTF

Conversation

@sherrylong

Copy link
Copy Markdown
Contributor

DO NOT MERGE

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Feb 19, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
site 5a55934 Commit Preview URL

Branch Preview URL
Feb 28 2026, 07:04 PM

Comment thread app/ctf/submit/page.tsx

@miguelaenlle miguelaenlle left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Love the overall design of this page! Here are some comments I have.

Also, I'd recommend you and Sherry test the full CTF flow like a participant would if you haven't already. I did not do that during this review.

Comment thread app/ctf/submit/page.tsx
Comment thread app/ctf/submit/page.tsx Outdated
Comment thread app/ctf/submit/page.tsx Outdated
Comment thread app/ctf/page.tsx Outdated
Comment thread app/ctf/submit/page.tsx Outdated

@miguelaenlle miguelaenlle left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'll approve once the merge conflicts are resolved.

@J164
J164 merged commit 9b5c667 into main Feb 28, 2026
2 checks passed
@J164
J164 deleted the CTF branch February 28, 2026 19:05
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.

5 participants