Skip to content

Multi check in - #162

Draft
aguilaair wants to merge 1 commit into
mainfrom
feat/multi-checkin
Draft

Multi check in#162
aguilaair wants to merge 1 commit into
mainfrom
feat/multi-checkin

Conversation

@aguilaair

@aguilaair aguilaair commented May 2, 2026

Copy link
Copy Markdown
Contributor

SUMMARY

Add multiple check in functionality.

CHANGED

  • Museum “Check in” button — Routes to the new check-in flow when there is no visit for today; if there is one for today, opens edit for that visit. Label switches between Check in and Edit today’s check-in.
  • createCheckIn / createMuseumCheckIn — Before insert (museum only), rejects duplicates for the same user, museum, and visit calendar day; returns a clear ConvexError message.
  • PostHog — checkin_button_clicked still reports isEditing for “editing today”; adds hasPastVisits.

ADDED

  • checkIns.visitCalendarDate (optional YYYY-MM-DD) — Stored on new museum check-ins; legacy rows omit it and dedup falls back to UTC date from visitDate / createdAt.
  • Convex helpers — utcYyyyMmDd, checkInCalendarKey, assertNoDuplicateMuseumVisitDay.
  • Mobile — [apps/mobile/lib/visit-calendar-date.ts](vscode-file://vscode-app/Applications/Cursor.app/Contents/ Resources/app/out/vs/code/electron-sandbox/workbench/apps/mobile/lib/visit-calendar-date.ts): formatLocalYyyyMmDd, utcYyyyMmDd, checkInCalendarKey.
  • createCheckIn arg — visitCalendarDate

DELETED

TESTING

ISSUE

#163

@vercel

vercel Bot commented May 2, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
aam-web Ready Ready Preview, Comment May 2, 2026 10:12pm

Request Review

@aguilaair aguilaair added the enhancement New feature or request label May 2, 2026
@aguilaair aguilaair self-assigned this May 2, 2026
@aguilaair aguilaair linked an issue May 2, 2026 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Multi Check in!

1 participant