Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added the announcement message type #1847

Merged
merged 16 commits into from
Apr 30, 2022

Conversation

mmattbtw
Copy link
Contributor

@mmattbtw mmattbtw commented Apr 16, 2022

Pull request checklist:

  • CHANGELOG.md was updated, if applicable
  • Documentation in docs/ or install-docs/ was updated, if applicable
  • I have tested all changes

Copy link
Member

@pajlada pajlada left a comment

Choose a reason for hiding this comment

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

Feature looks good to me, happy to add it. I think we should call it announce instead though, if that's something you can do just poke me when it's done and I'll review&merge this in. If not, let me know and I'll make those changes on your branch 👍 Thank you!

pajbot/bot.py Outdated Show resolved Hide resolved
mmattbtw and others added 2 commits April 23, 2022 10:19
Co-authored-by: pajlada <rasmus.karlsson@pajlada.com>
@mmattbtw mmattbtw requested a review from pajlada April 23, 2022 15:24
templates/admin/create_command.html Outdated Show resolved Hide resolved
templates/admin/create_timer.html Outdated Show resolved Hide resolved
@mmattbtw mmattbtw requested a review from pajlada April 24, 2022 18:14
CHANGELOG.md Outdated Show resolved Hide resolved
pajbot/bot.py Outdated Show resolved Hide resolved
pajbot/bot.py Outdated Show resolved Hide resolved
mmattbtw and others added 3 commits April 27, 2022 15:23
Co-authored-by: ALazyMeme <12804673+ALazyMeme@users.noreply.github.com>
@mmattbtw mmattbtw requested a review from ALazyMeme April 27, 2022 20:25
pajbot/modules/quest.py Outdated Show resolved Hide resolved
pajbot/modules/quest.py Outdated Show resolved Hide resolved
mmattbtw and others added 2 commits April 28, 2022 08:17
Co-authored-by: ALazyMeme <12804673+ALazyMeme@users.noreply.github.com>
Co-authored-by: ALazyMeme <12804673+ALazyMeme@users.noreply.github.com>
@mmattbtw mmattbtw requested a review from ALazyMeme April 28, 2022 20:02
Copy link
Member

@ALazyMeme ALazyMeme left a comment

Choose a reason for hiding this comment

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

yep

@ALazyMeme ALazyMeme changed the title added the announcement message type Added the announcement message type Apr 30, 2022
@ALazyMeme ALazyMeme merged commit 5f3c416 into pajbot:master Apr 30, 2022
@mmattbtw mmattbtw deleted the new/announcement-msg-type branch May 1, 2022 14:47
pajlada added a commit that referenced this pull request Jun 4, 2022
This was missing since the `announce` message type was added in #1847
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.

None yet

3 participants