Skip to content

Open onboarding meeting poll during global team onboarding#77

Merged
drmowinckels merged 1 commit into
mainfrom
feat/onboarding-meeting-poll
Jul 3, 2026
Merged

Open onboarding meeting poll during global team onboarding#77
drmowinckels merged 1 commit into
mainfrom
feat/onboarding-meeting-poll

Conversation

@drmowinckels

Copy link
Copy Markdown
Member

Summary

Global team onboarding now sets up the onboarding meeting automatically instead of leaving a manual Calendly checklist item.

  • New exported gt_schedule_onboarding_meeting() opens a samkoma "find a time" poll and posts the poll link as a comment on the onboarding issue, all under Jinx's identity.
  • The poll offers a two-week window of candidate dates starting one week after creation (Sys.Date() + 7), giving the new member and team lead time to paint availability before the first possible slot. Daily band defaults to a broad UTC window (08:00–20:00, 30-min slots) since the global team spans timezones; all overridable.
  • gt_finalize_onboarding() calls the scheduler after creating the issue. A samkoma outage warns and is skipped rather than aborting onboarding — the issue is still created.
  • The onboarding template's stale [onboarding meeting](calendly…) step is replaced with a note to pick a slot from the poll Jinx posts.

Reuses the existing meeting_poll_create() / meeting_poll_format_created() / announce_post_reply() primitives; api.samkoma.org (the live production host, v1 API) is the default and needs no env var. Full suite passes (797 tests, 0 failures); NEWS.md updated.

Global team onboarding now sets up the onboarding meeting automatically.
gt_finalize_onboarding() calls the new gt_schedule_onboarding_meeting(),
which opens a samkoma "find a time" poll spanning a two-week window of
candidate dates starting one week after creation, and posts the poll link
as a comment on the onboarding issue. Both run under Jinx's identity.

A samkoma outage warns and is skipped rather than aborting onboarding,
and the onboarding template's stale manual Calendly step is replaced with
a note to pick a slot from the poll Jinx posts.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@jinx-familiar

jinx-familiar Bot commented Jul 1, 2026

Copy link
Copy Markdown

🎉 Good Practice Check

All checks passed!

Updated: 2026-07-01T19:04:52.236Z

@drmowinckels drmowinckels merged commit e17a5ce into main Jul 3, 2026
4 checks passed
@drmowinckels drmowinckels deleted the feat/onboarding-meeting-poll branch July 3, 2026 17:38
drmowinckels added a commit that referenced this pull request Jul 4, 2026
Resolve conflicts from #77/#79/#80:
- airtable-sync.R: keep only the shared helpers (airtable_list_records,
  airtable_extract_photo); main removed gt_sync_airtable and this branch
  moved the directory functions into directory-airtable*.R.
- ops-airtable-sync.yml: removed. Global-team sync is gone (gt_sync
  removed on main) and the directory sync now runs from the private
  directory repo, so the public workflow has nothing left to do.
- NEWS.md: combine both changelogs.
- Rename this branch's normalize_handle(x, prefixes) helper to
  strip_handle_prefixes; main added its own normalize_handle(x) in
  authorize.R (name collision that broke authorization).
- Drop this branch's duplicate is_blank; reuse the existing one in
  rag-chunk.R.

Merged suite: 1023 pass, 0 fail/error. R CMD check clean.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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.

1 participant