Skip to content

Checked In Status in Registration - #227

Merged
12458 merged 9 commits into
mainfrom
sse/check-in
Jul 15, 2025
Merged

Checked In Status in Registration#227
12458 merged 9 commits into
mainfrom
sse/check-in

Conversation

@12458

@1245812458 commented Jul 10, 2025

Copy link
Copy Markdown
Contributor

Resolves#224 and #225

@12458
12458 requested a review from ganning127July 10, 2025 22:30
github-advanced-security[bot]
github-advanced-securityAI found potential problems Jul 10, 2025
View reviewed changes
Comment threadservices/hexathons/src/routes/interaction.ts Fixed
@12458@github-advanced-security
Potential fix for code scanning alert no. 138: Use of externally-cont…
5ea4b48
…rolled format string
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
ganning127
ganning127 reviewed Jul 12, 2025
View reviewed changes
Comment threadservices/registration/src/routes/statistics.ts Outdated
ganning127
ganning127 reviewed Jul 12, 2025
View reviewed changes
Comment threadservices/registration/src/routes/statistics.ts Outdated
ganning127
ganning127 reviewed Jul 12, 2025
View reviewed changes
Comment threadservices/registration/src/routes/application.ts Outdated
ganning127
ganning127 reviewed Jul 12, 2025
View reviewed changes
Comment threadservices/hexathons/src/routes/interaction.ts
ganning127
ganning127 requested changes Jul 12, 2025
View reviewed changes

@ganning127ganning127 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.

Good start @12458! Just left some comments.

We also need an endpoint to check whether a user is checked in or not, so we can show it on the applications page.

@12458

12458 commented Jul 12, 2025

Copy link
Copy Markdown
ContributorAuthor

Added endpoint to check checked in status

@12458
12458 requested a review from ganning127July 12, 2025 20:27
ganning127
ganning127 approved these changes Jul 13, 2025
View reviewed changes

@ganning127ganning127 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.

LGTM @12458!

@12458
12458 merged commit 472fb8b into mainJul 15, 2025
3 checks passed
@12458
12458 deleted the sse/check-in branch July 15, 2025 00:08
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ganning127ganning127ganning127 approved these changes
@lyangji1011lyangji1011Awaiting requested review from lyangji1011

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Create a new application StatusType of Checked-In

3 participants

@12458@github-advanced-security@ganning127