Skip to content

feat: send Discord notifications for waitlist and new Google signups#3

Merged
anxi01 merged 1 commit into
mainfrom
codex/waitlist-discord-webhook
May 13, 2026
Merged

feat: send Discord notifications for waitlist and new Google signups#3
anxi01 merged 1 commit into
mainfrom
codex/waitlist-discord-webhook

Conversation

@anxi01

@anxi01 anxi01 commented May 13, 2026

Copy link
Copy Markdown
Member

Summary

  • send a best-effort Discord webhook after a waitlist signup is saved
  • format waitlist Discord embeds with the user, submitted video link, signup time, and YouTube thumbnail preview
  • send a separate best-effort Discord webhook when a user signs in with Google for the first time

Details

  • adds DISCORD_WAITLIST_WEBHOOK_URL for waitlist notifications
  • adds DISCORD_NEW_USER_WEBHOOK_URL for first-time Google signup notifications
  • keeps both webhook paths non-blocking so API success does not depend on Discord delivery
  • documents the new env vars in .env.example and README.md

Testing

  • python3 -m pytest -q

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 54d81a7394

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread app/services/discord.py


def build_waitlist_signup_payload(entry: WaitlistEntry) -> dict:
timestamp = int(entry.created_at.astimezone(UTC).timestamp())

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Treat naive created_at as UTC before converting timestamp

entry.created_at.astimezone(UTC) produces incorrect epoch values when created_at is naive (common with SQLite and some DB/driver timezone settings), because Python assumes the host local timezone for naive datetimes. In non-UTC deployments this shifts the displayed Discord "Signed Up" time by the server offset, so notifications show wrong signup times even though the stored value represents UTC.

Useful? React with 👍 / 👎.

@anxi01 anxi01 self-assigned this May 13, 2026
@anxi01 anxi01 merged commit 0ae8682 into main May 13, 2026
2 checks passed
@anxi01 anxi01 deleted the codex/waitlist-discord-webhook branch May 13, 2026 13:49
@anxi01 anxi01 changed the title feat: send Discord waitlist notifications with video previews feat: send Discord notifications for waitlist and new Google signups May 13, 2026
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