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

Event cleanup #31

Merged
merged 9 commits into from
Jul 7, 2018
Merged

Event cleanup #31

merged 9 commits into from
Jul 7, 2018

Conversation

JulianWebb
Copy link
Contributor

Some basic cleanup of the event files, moved the embed functions back into extraneous.js, changed how arguments work for plugins, and made the command embed show up for commands from other servers.

@JulianWebb JulianWebb self-assigned this Jul 7, 2018
@JulianWebb JulianWebb requested a review from isaacmorneau July 7, 2018 18:10
@JulianWebb
Copy link
Contributor Author

This PR should fix #21 and kinda fix #22

Copy link
Contributor

@isaacmorneau isaacmorneau left a comment

Choose a reason for hiding this comment

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

i dont even know what this does but it seems legit

@isaacmorneau isaacmorneau merged commit 7116b7a into master Jul 7, 2018
@isaacmorneau isaacmorneau deleted the event-cleanup branch July 7, 2018 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants