Skip to content

feat(place/vip_visitor_automation): add VIP visitor workflow - #650

Open
KesterJJ wants to merge 1 commit into
masterfrom
feat/new-driver/place/vip-visitor-automation
Open

KesterJJ wants to merge 1 commit into
masterfrom
feat/new-driver/place/vip-visitor-automation

Conversation

@KesterJJ

Copy link
Copy Markdown
Contributor

Adds VIP visitor automation for standalone vip-visitor bookings. The driver emails the assistant an invitation with a QR code and confirmation link, emails the submitted details to configured VIP handlers, and creates a Planner plan through the existing Calendar driver after explicit confirmation.

Creates To Do, In Progress and Done buckets and up to nine tasks based on the selected services. Supports current AECB form field aliases, configurable task assignments, optional arrival/departure notices, duplicate suppression, refreshed invitations after edits, and checkpointed recovery from partial Planner failures.

Validation

  • Driver harness: docker exec placeos-drivers report drivers/place/vip_visitor_automation.cr — 1 tested, 0 failures, 0 timeouts.
  • Specs cover invitation/JWT claims, explicit acceptance, booking filtering, all conditional tasks, field aliases, duplicate events, partial-failure recovery, invitation edits and lifecycle notices.
  • Crystal formatting and diff checks passed. Tests use mocked services; no live email or Microsoft Graph resources were created.

Integration requirements

  • The external confirmation app needs a scoped backend endpoint and must save extension_data.vip_confirmed: true. Existing guest tokens cannot PATCH the general booking endpoint.
  • Planner uses an existing Microsoft 365 group; this driver does not provision a Team. Grouped child bookings are excluded.
  • The README documents settings, templates, permissions and recovery limits. The companion VisitorMailer exclude_booking_types change is outside this new-driver-only PR, following the ai-env-drivers contribution process; deployments using both mail workflows need that exclusion support.

@KesterJJ
KesterJJ requested review from stakach and w-le November 12, 2026 08:12

@w-le w-le 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.

Seems like it would be a good idea to add a spec for the visitor_planner logic driver right? Unless the testing of that is already covered by the upstream logic driver's spec.

Other than that. Looks good to e2e test on a dev or uat server, assuming it compiles?

This branch has not been deployed

No deployments
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.

3 participants