Skip to content

Minimal features - #82

Open
huncijr wants to merge 9 commits into
hackclub:mainfrom
huncijr:minimal_features
Open

Minimal features#82
huncijr wants to merge 9 commits into
hackclub:mainfrom
huncijr:minimal_features

Conversation

@huncijr

@huncijr huncijr commented Jun 19, 2026

Copy link
Copy Markdown

Added 4 features : cancelled event badge, an interest count display showing how many people RSVP'd via Slack,(For the interest count to work with real RSVP data from the website, Isabelle needs a non-read-only REST endpoint I already fixed that on: hackclub/isabelle#25). Add a POST /api/rsvp route in app.py that calls the existing toggle_user_interest() from database.py, accepts { event_id, user_id } in the request body, and returns the updated interest count. The internal logic is already complete .It just needs to be exposed as a public endpoint so the Events website can POST RSVP actions directly instead of relying only on Slack reactions.) and expanded YouTube recording support beyond AMA-only events. The data.js mapping was extended to include cancelled, rawCancellation, and interestCount fields. And also added an icon for light mode :)
Screenshot From 2026-06-19 14-19-50
image
Screenshot From 2026-06-19 14-45-40

@vercel

vercel Bot commented Jun 19, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
events Error Error Jun 19, 2026 12:54pm

Request Review

@vercel

vercel Bot commented Jun 19, 2026

Copy link
Copy Markdown

Deployment failed with the following error:

The `vercel.json` schema validation failed with the following message: should NOT have additional property `public`

Learn More: https://vercel.com/docs/concepts/projects/project-configuration

@vercel

vercel Bot commented Jun 19, 2026

Copy link
Copy Markdown

@huncijr is attempting to deploy a commit to the Hack Club Team on Vercel.

A member of the Team first needs to authorize it.

@huncijr

huncijr commented Jul 12, 2026

Copy link
Copy Markdown
Author

Will this be reviewed?

@v1ctorio

Copy link
Copy Markdown
Member

@huncijr can you resolve the conflicts? The RSVP via web already is implemented.
Also, is this UI change intentional?

old new
image image

imho, I don't see any reason to make the borders wider, but if it's not-intentional, it should be fixed.

I do like the cancellation info banner tho! Cool feature.

@huncijr

huncijr commented Jul 15, 2026

Copy link
Copy Markdown
Author

Hi, I fixed it!

@v1ctorio

Copy link
Copy Markdown
Member

Alright, will look into it

@v1ctorio

Copy link
Copy Markdown
Member

This does not build.

@huncijr

huncijr commented Jul 17, 2026

Copy link
Copy Markdown
Author

I hope everything's okay now, the problem was that I didn't accidentally pull, and it caused the merge conflicts.

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