Skip to content

feat: Mirror status updates onto discord#99

Merged
hrideshmg merged 2 commits intoamfoss:developfrom
Khushal928:main
Mar 6, 2026
Merged

feat: Mirror status updates onto discord#99
hrideshmg merged 2 commits intoamfoss:developfrom
Khushal928:main

Conversation

@Khushal928
Copy link
Member

This PR adds a new task that sends all the updates that were sent last day to discord channels

  • If member's track is "Inductee", updates will be sent to their group status update channels.
  • Other track updates will be sent to respective track channels.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new scheduled task that mirrors daily status-update emails into the appropriate Discord channels based on member track/group, wiring in the needed GraphQL fields and channel IDs.

Changes:

  • Introduces MirrorNewUpdates task to fetch status-update emails via IMAP and post them to Discord channels.
  • Extends GraphQL member fetching/model to include email (and groupId in the model) to route updates correctly.
  • Adds per-track and per-group status-update channel IDs plus new mail/IMAP dependencies and env sample entries.

Reviewed changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
src/tasks/status_update_mirror.rs New scheduled task that reads status-update emails and posts them to Discord with track/group routing.
src/tasks/mod.rs Registers the new mirror task for scheduling.
src/tasks/status_update.rs Removes an unused import.
src/graphql/queries.rs Adds email to the member query payload.
src/graphql/models.rs Adds email and group_id fields to Member for routing logic.
src/ids.rs Adds channel ID constants for track/group-specific status update channels.
Cargo.toml Adds IMAP/mail parsing dependencies.
Cargo.lock Lockfile updates for added dependencies.
.env.sample Adds EMAIL_ID and APP_PASSWORD sample variables.
src/config.rs Minor formatting change for AMD_API_KEY env lookup.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@amfoss amfoss deleted a comment from Copilot AI Mar 6, 2026
@amfoss amfoss deleted a comment from Copilot AI Mar 6, 2026
@amfoss amfoss deleted a comment from Copilot AI Mar 6, 2026
@amfoss amfoss deleted a comment from Copilot AI Mar 6, 2026
@Khushal928 Khushal928 requested a review from hrideshmg March 6, 2026 10:10
hrideshmg
hrideshmg previously approved these changes Mar 6, 2026
Copy link
Member

@hrideshmg hrideshmg left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for the PR, just tested it and it seems to be working well.

Will merge after pushing a small commit to rename the env vars to follow bedrock conventions.

@hrideshmg hrideshmg merged commit 6c50907 into amfoss:develop Mar 6, 2026
2 checks passed
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