Skip to content

Implement DM to applicants on induction#465

Merged
MattyTheHacker merged 5 commits intomainfrom
459-send-induction-dm-to-applicants
Apr 18, 2025
Merged

Implement DM to applicants on induction#465
MattyTheHacker merged 5 commits intomainfrom
459-send-induction-dm-to-applicants

Conversation

@MattyTheHacker
Copy link
Copy Markdown
Member

No description provided.

@MattyTheHacker MattyTheHacker added the enhancement New feature or request label Apr 15, 2025
@MattyTheHacker MattyTheHacker self-assigned this Apr 15, 2025
@MattyTheHacker MattyTheHacker linked an issue Apr 15, 2025 that may be closed by this pull request
@MattyTheHacker
Copy link
Copy Markdown
Member Author

not tested yet

Copy link
Copy Markdown
Contributor

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

This PR implements a direct message (DM) to applicants upon induction, providing them with useful onboarding instructions. The changes include:

  • Sending a DM with induction instructions to the new applicant.
  • Handling potential discord.Forbidden errors when sending the DM.
  • Sending an ephemeral follow-up message to confirm the applicant’s role update.

Comment thread cogs/make_applicant.py Outdated
Comment thread cogs/make_applicant.py Outdated
Comment thread cogs/make_applicant.py Outdated
Comment thread cogs/make_applicant.py Outdated
Comment thread cogs/make_applicant.py Outdated
Comment thread cogs/make_applicant.py Outdated
Comment thread cogs/make_applicant.py Outdated
Comment thread cogs/make_applicant.py Outdated
@MattyTheHacker MattyTheHacker force-pushed the 459-send-induction-dm-to-applicants branch from 63be0fd to b3487e5 Compare April 18, 2025 19:22
Copy link
Copy Markdown
Member

@CarrotManMatt CarrotManMatt left a comment

Choose a reason for hiding this comment

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

Sorry.🫣

Comment thread cogs/make_applicant.py Outdated
Co-authored-by: Matt Norton <matt@carrotmanmatt.com>
@MattyTheHacker MattyTheHacker merged commit f18fc22 into main Apr 18, 2025
9 checks passed
@MattyTheHacker MattyTheHacker deleted the 459-send-induction-dm-to-applicants branch April 18, 2025 19:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement DM'ing a user when they are made an applicant

3 participants