Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/confirm attendance non hacker #347

Closed
wants to merge 1 commit into from

Conversation

njhuey
Copy link
Contributor

@njhuey njhuey commented Jan 25, 2024

Resolves #322.

Implements logic to check in non-hackers.

Copy link
Contributor

github-actions bot commented Jan 25, 2024

Deploy preview for irvinehacks-site-2024 ready!

Name IrvineHacks 2024 Site
Preview Visit Preview
Commit 5bba534

@njhuey njhuey force-pushed the feature/confirm-attendance-non-hacker branch from 2ebc003 to 5bba534 Compare January 25, 2024 03:59
@taesungh
Copy link
Member

We'll still need to check attending status on non-hackers, so I think most of the changes here are unnecessary. One thing we do need to add is checking that participants have a non-empty role, but other than that, I think we can just adapt the existing check-in function.

@taesungh
Copy link
Member

Apologies for the confusion, I'll update the description on #322, please work on #355 instead.

@taesungh taesungh closed this Jan 25, 2024
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.

[API] Add Admin endpoint for directors to confirm attendance of a non-hacker
2 participants