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

Discord Bee #239

Merged
merged 4 commits into from
May 8, 2019
Merged

Discord Bee #239

merged 4 commits into from
May 8, 2019

Conversation

CalmBit
Copy link
Collaborator

@CalmBit CalmBit commented May 4, 2019

Adds the discord bee! (Tackling #235 )

Unfortunately, it requires a bit of wrangling (a user has to create a discord app, create a bot on that app, get its API token, and use that to seed the bot) - this is all unavoidable by Discord's design, however. Everything else is fluid!

Copy link
Owner

@muesli muesli left a comment

Choose a reason for hiding this comment

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

Left a few remarks & questions. Looking good otherwise!

bees/discordbee/discordbee.go Outdated Show resolved Hide resolved
bees/discordbee/discordbee.go Outdated Show resolved Hide resolved
bees/discordbee/discordbee.go Show resolved Hide resolved
bees/discordbee/discordbeefactory.go Outdated Show resolved Hide resolved
bees/discordbee/discordbeefactory.go Outdated Show resolved Hide resolved
bees/discordbee/discordbee.go Outdated Show resolved Hide resolved
bees/discordbee/discordbee.go Outdated Show resolved Hide resolved
bees/discordbee/discordbeefactory.go Outdated Show resolved Hide resolved
.vscode/settings.json Outdated Show resolved Hide resolved
@muesli muesli merged commit 71b73db into muesli:master May 8, 2019
Mark-Jung pushed a commit to Mark-Jung/beehive that referenced this pull request May 15, 2019
@gingerbeardman
Copy link

A step by step guide for this would be awesome!

I have created the discord app and bot and have my token, and i've created my bees and chains but something is obviously missing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants