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

feat(notification): add discord groups #1211

Merged
merged 13 commits into from
Dec 9, 2020
Merged

feat(notification): add discord groups #1211

merged 13 commits into from
Dec 9, 2020

Conversation

styler2go
Copy link
Contributor

Description

I have added the option to select specific discord groups for each series. I made it because it was requested in my discord. If you don't need it feel free to decline/close this request

@styler2go styler2go requested a review from jef as a code owner December 6, 2020 08:18
Copy link
Owner

@jef jef left a comment

Choose a reason for hiding this comment

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

This is great! I know I've been asked for this a few times now!

If you could just make the documentation change, that would be great.


I think this is also another reason why I have been working on https://github.com/jef/streetmerchant/tree/prerelease/4.0.0.

Config drift is hellish 🙃

docs/reference/notification.md Outdated Show resolved Hide resolved
@styler2go styler2go requested a review from jef December 7, 2020 14:00
Copy link
Owner

@jef jef left a comment

Choose a reason for hiding this comment

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

Just a couple more things and we'll be there!

Thank you very much!!!

src/notification/discord.ts Outdated Show resolved Hide resolved
docs/reference/notification.md Outdated Show resolved Hide resolved
src/notification/discord.ts Outdated Show resolved Hide resolved
@styler2go styler2go requested a review from jef December 9, 2020 08:30
Copy link
Owner

@jef jef left a comment

Choose a reason for hiding this comment

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

Awesome! Thank you very much for all your work!

@jef jef changed the title feat: discord notification groups feat(notification): add discord groups Dec 9, 2020
@jef jef merged commit 2632386 into jef:main Dec 9, 2020
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.

2 participants