Skip to content

Conversation

@christianhelp
Copy link
Collaborator

Why

During setup with Sunhacks, we had noticed some small bugs and received feedback on some fixes that we should make. This PR satisfies those

What

Fixed the following issues:

  • Made the cloudflare bucket name into an environment variable since the bucket name should be different and needs to be configured
  • Updated the permissions to allow volunteer see admin. This one should be redone later on along with a revamp of the permissions system, but for the sake of this PR, it will be added.
  • Fixed registration form bug when saving data
  • Debounced registration form data save

Satisfies

#184

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jul 3, 2025

Deploying hackkit with  Cloudflare Pages  Cloudflare Pages

Latest commit: a7aeb39
Status: ✅  Deploy successful!
Preview URL: https://817813b1.hackkit.pages.dev
Branch Preview URL: https://feedback-fixes.hackkit.pages.dev

View logs

@cloudflare-workers-and-pages
Copy link

Deploying with  Cloudflare Workers  Cloudflare Workers

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

Status Name Latest Commit Updated (UTC)
❌ Deployment failed
View logs
backups-test 84f8646 Jul 03 2025, 12:55 AM

1 similar comment
@cloudflare-workers-and-pages
Copy link

Deploying with  Cloudflare Workers  Cloudflare Workers

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

Status Name Latest Commit Updated (UTC)
❌ Deployment failed
View logs
backups-test 84f8646 Jul 03 2025, 12:55 AM

@joshuasilva414 joshuasilva414 self-requested a review July 14, 2025 23:25
Copy link
Contributor

@joshuasilva414 joshuasilva414 left a comment

Choose a reason for hiding this comment

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

Looks solid to me

@joshuasilva414 joshuasilva414 merged commit 36fe615 into dev Jul 27, 2025
2 checks passed
team-sunhacks added a commit to sunhacksio/sunhacks.io-25 that referenced this pull request Aug 6, 2025
* updates infisical json

* Feedback fixes (acmutsa#185)

* updates to use bucket-env

* fix volunteer permissions

* fix bug with local storage write

* debounce registration write

* remove create registration alert

* adds volunteer action

* removes points

* updates ui for volunteers

* removes super admin actions

* updates event ui for volunteers

* adds recent registrations

* formatter

* Create LICENSE

* Remove Unimplemented Sections  (acmutsa#190)

* Clearing out admin "coming soon" and some of the unused config variables

* Cleared out Secret Registration sections

* Commented out all code that related to teams and invites

* Deleted all issues with unimplemented sections fully

* Hero Page Update alpha-1

---------

Co-authored-by: Christian Walker <christiandwalker2021@gmail.com>
Co-authored-by: Christian Walker <76548772+christianhelp@users.noreply.github.com>
Co-authored-by: Joshua Silva <72359611+joshuasilva414@users.noreply.github.com>
Co-authored-by: Ryan Godson Mathuram <168239412+ryanMathram@users.noreply.github.com>
Co-authored-by: Ansh Bhalala <ansh@bhalala.org>
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