Skip to content

Add optional Campus Expert role tag to blog digest - #10

Open
MetzinAround wants to merge 1 commit into
mainfrom
copilot/blog-role-mention
Open

MetzinAround wants to merge 1 commit into
mainfrom
copilot/blog-role-mention

Conversation

@MetzinAround

Copy link
Copy Markdown
Contributor

Copilot Generated

Summary

  • Add optional blog.mention_role_id to tag @campus Expert on the existing blog digest message.
  • Allow only the configured role to be notified. YouTube does not tag anyone.
  • Leave empty results, repeated runs, errors, and SparkedHost's scheduled start-and-stop behavior unchanged.
  • Document the existing Discord permissions and SparkedHost setup. No new dependencies or saved records.

Checks

  • Python compile check passed.
  • All 20 unit tests passed, including tagged and untagged messages, invalid role settings, empty blog results, and HTTP errors.
  • Discord calls were mocked. No live notifications were sent.

Deployment

blog.mention_role_id is deliberately null until the actual Campus Expert role ID is supplied. Copy the role's numeric ID in Discord and set it in quotes on SparkedHost after deploying.

Keep the existing schedule, python main.py beginning command, and secrets. Confirm the bot's existing mention permission applies in the destination channel. Coordinate one live check after deployment because it will notify real members.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: a035a73f-22b4-4443-927e-bbf9bf05f05f
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