Skip to content

implement solo attendance scanner.#3

Merged
vijay-sb merged 20 commits into
mainfrom
feat/scanner-page
Feb 10, 2026
Merged

implement solo attendance scanner.#3
vijay-sb merged 20 commits into
mainfrom
feat/scanner-page

Conversation

@KiranRajeev-KV

Copy link
Copy Markdown
Member
  • Initial trial
  • Add login page
  • address PR review comments
  • Initial events page
  • Update events page, sidebar and routes
  • feat: add attendance schema for marking attendance
  • feat: add Toaster component for notifications
  • feat: add MARK_ATTENDANCE endpoint for solo attendance
  • feat: implement Solo Attendance Scanner component for real-time attendance marking
  • feat: add ScannerPage component for solo attendance scanning

MinotiGupta
MinotiGupta previously approved these changes Feb 10, 2026

@MinotiGupta MinotiGupta left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@KiranRajeev-KV KiranRajeev-KV dismissed MinotiGupta’s stale review February 10, 2026 14:40

The merge-base changed after approval.

MinotiGupta
MinotiGupta previously approved these changes Feb 10, 2026

@MinotiGupta MinotiGupta left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@KiranRajeev-KV KiranRajeev-KV dismissed MinotiGupta’s stale review February 10, 2026 14:46

The merge-base changed after approval.

@IAmRiteshKoushik

Copy link
Copy Markdown
Member

So, the merge base keeps changing ?

@vijay-sb

Copy link
Copy Markdown
Member

So, the merge base keeps changing ?

whats happening?

@IAmRiteshKoushik

IAmRiteshKoushik commented Feb 10, 2026

Copy link
Copy Markdown
Member

So, the merge base keeps changing ?

whats happening?

Kiran is dying. His PR is not getting merged. (even though we approved)

@vijay-sb

Copy link
Copy Markdown
Member

So, the merge base keeps changing ?

whats happening?

Kiran is dying. His PR is not getting merged. (even though we approved)

lol

@IAmRiteshKoushik

Copy link
Copy Markdown
Member

So, the merge base keeps changing ?

whats happening?

Kiran is dying. His PR is not getting merged. (even though we approved)

lol

Pretty sure he did smtg stupid on the main branch is now dying hopelessly on this PR

@vijay-sb

Copy link
Copy Markdown
Member

So, the merge base keeps changing ?

whats happening?

Kiran is dying. His PR is not getting merged. (even though we approved)

lol

Pretty sure he did smtg stupid on the main branch is now dying hopelessly on this PR

skill issuess frrr

@vijay-sb vijay-sb merged commit 8c51b15 into main Feb 10, 2026
2 checks passed
@IAmRiteshKoushik

IAmRiteshKoushik commented Feb 10, 2026

Copy link
Copy Markdown
Member

So, the merge base keeps changing ?

whats happening?

Kiran is dying. His PR is not getting merged. (even though we approved)

lol

Pretty sure he did smtg stupid on the main branch is now dying hopelessly on this PR

skill issuess frrr

Finally merged it huh ? What was the soln ?

@vijay-sb

Copy link
Copy Markdown
Member

So, the merge base keeps changing ?

whats happening?

Kiran is dying. His PR is not getting merged. (even though we approved)

lol

Pretty sure he did smtg stupid on the main branch is now dying hopelessly on this PR

skill issuess frrr

Finally merged it huh ?

he removed branch protection. mid!

@IAmRiteshKoushik

Copy link
Copy Markdown
Member

So, the merge base keeps changing ?

whats happening?

Kiran is dying. His PR is not getting merged. (even though we approved)

lol

Pretty sure he did smtg stupid on the main branch is now dying hopelessly on this PR

skill issuess frrr

Finally merged it huh ?

he removed branch protection. mid!

Wait, if there's on branch protection, then can't he just merge this branch locally and push it upstream ? @KiranRajeev-KV dummy dum dum.

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.

4 participants