Skip to content

Add the MonthlyMeetupDispatch workflow - #3

Open
MasonEgger wants to merge 1 commit into
mainfrom
meetup-dispatch-workflow
Open

Add the MonthlyMeetupDispatch workflow#3
MasonEgger wants to merge 1 commit into
mainfrom
meetup-dispatch-workflow

Conversation

@MasonEgger

Copy link
Copy Markdown
Member

Summary

First workflow under the repo's new identity as pytexas/dispatch, the communications dispatch service.

  • New meetup_dispatch package: MonthlyMeetupDispatch creates the Discord scheduled event, then posts the asset handoff to the marketing webhook and the setup summary to the organizers webhook.
  • The Discord event is external with location text "PyTexas Stage" (stage-linked events have the stage audio bug; a comment guards against "fixing" it).
  • Missing PYTEXAS_DISCORD_BOT_TOKEN degrades gracefully: both messages still post with a TBD event link and the result carries a note.
  • Inputs are links only (claim check via Drive) to stay under Temporal's 2MB payload cap.
  • Tests written first: 7 passing (workflow chain, message content, token-missing skip path, formatting), mypy strict clean on the new code.

Notes

  • Pre-existing mypy failures in tests/test_pretix.py exist on main and are not touched here, so just check fails on typecheck before and after this PR.
  • The deployed env needs PYTEXAS_MARKETING_WEBHOOK, PYTEXAS_MEETUP_WEBHOOK, and optionally PYTEXAS_DISCORD_BOT_TOKEN added to the infrastructure repo's sops env for this service.
  • The infrastructure repo still references the old repo name; rename PR coming separately.

The repo is now pytexas/dispatch, the general communications dispatch service.
A new meetup_dispatch package holds the workflow that creates the external Discord scheduled event ("PyTexas Stage" location, never stage-linked due to the audio bug), then posts the asset handoff to the marketing webhook and the setup summary to the organizers webhook.
Event creation degrades gracefully: without PYTEXAS_DISCORD_BOT_TOKEN the messages still post with a TBD link and the result carries a note.
Inputs are links only to stay under Temporal's 2MB payload cap.
The existing worker registers both workflows; README covers the new workflow, env vars, and the CLI start example.
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