Skip to content

route status updates to group channels - #107

Open
Khushal928 wants to merge 9 commits into
amfoss:developfrom
Khushal928:main
Open

route status updates to group channels#107
Khushal928 wants to merge 9 commits into
amfoss:developfrom
Khushal928:main

Conversation

@Khushal928

Copy link
Copy Markdown
Member

route status updates to group channels instead of tracks

Copilot AI lite review requested due to automatic review settings August 8, 2026 18:39

Copilot AI 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.

Pull request overview

This PR updates the status-update mirroring task so Discord status updates are routed to group-specific channels (based on group_id) instead of track-specific channels.

Changes:

  • Simplifies channel routing logic to select a destination channel by group (1–4), with a fallback to the default status channel.
Suppressed comments (1)

src/tasks/status_update_mirror.rs:181

  • After removing the track-based routing arms, AI_STATUS_UPDATE_CHANNEL_ID, WEB_STATUS_UPDATE_CHANNEL_ID, MOBILE_STATUS_UPDATE_CHANNEL_ID, and SYSTEMS_STATUS_UPDATE_CHANNEL_ID appear to be unused in this file (only imported). Please remove these from the use crate::ids::{...} list to avoid unused-import warnings and keep the import list accurate.
    let channel_id = match (track.as_str(), group) {
        (_, 1) => GROUP_ONE_STATUS_UPDATE_CHANNEL_ID,
        (_, 2) => GROUP_TWO_STATUS_UPDATE_CHANNEL_ID,
        (_, 3) => GROUP_THREE_STATUS_UPDATE_CHANNEL_ID,
        (_, 4) => GROUP_FOUR_STATUS_UPDATE_CHANNEL_ID,

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/tasks/status_update_mirror.rs
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.

5 participants