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.

Sign up for GitHub

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

Jump to bottom

Multi-server configuration #26

Merged
merged 54 commits into from
Jun 28, 2019
Merged

Multi-server configuration #26

merged 54 commits into from
Jun 28, 2019

Conversation

s0lst1ce
Copy link
Collaborator

@s0lst1ce s0lst1ce commented May 28, 2019 •
edited
Loading

The point of this PR is to bring multi-server support to the bot. The first and main step for this is to allow the owners of the servers to configure the way the bot works in their server. This should complete the 0.1 milestone.
Here's a list of all that has to be done:

@s0lst1ce s0lst1ce self-assigned this May 30, 2019
@s0lst1ce s0lst1ce added bug Something isn't working quality all issues which only affect the way code is written and that won't affect functionnality enhancement New feature or request labels May 30, 2019
@s0lst1ce s0lst1ce added this to the 0.1 milestone May 30, 2019
@s0lst1ce s0lst1ce marked this pull request as ready for review May 30, 2019 09:38
@s0lst1ce s0lst1ce mentioned this pull request May 30, 2019
Closed
@s0lst1ce
added basic Roll configuration
8c57b58
@s0lst1ce
added server-specific Slapping, fixing #22
cc28a65
@s0lst1ce
cleaned some loosy syntax
175c501
@s0lst1ce
added partial role configuration option
7ce262b
@s0lst1ce
added file modification time utilities
d00ae4c
@s0lst1ce
changed Config to use the new configuration structure (see #4 )
5477a83
@s0lst1ce
added welcome/goodbye messages configuration
426d142
@s0lst1ce
advertisement configuration
0fd7e7b
@s0lst1ce
perfected BotEssentials integreation with the new infrstrcture (fixing …
7f22e13 
…#30 and giving a workaround for #29 )
@s0lst1ce
Poll: removed uncessary code & wrote new to fit new infrastructure
53f7af2
@s0lst1ce
Role: removed uncessary code & wrote new to fit new infrastructure
3757493
@s0lst1ce
updated documentation
abadb92
@s0lst1ce
added per-server permission check & small bugfixes
28d1244
@s0lst1ce
improved Slapping
13fc2ed
@s0lst1ce
adapted Role to the new structure
b0de62a
@s0lst1ce s0lst1ce mentioned this pull request Jun 11, 2019
Closed
@s0lst1ce
Copy link
Collaborator Author

Well this PR is done and marks the end of the v1 milestone. Look at the release for more information. Great job anyways. We've gotten over the 1K cloc lines of code and this PR added 1.5K changes.

@s0lst1ce s0lst1ce closed this Jun 28, 2019
@s0lst1ce s0lst1ce reopened this Jun 28, 2019
@s0lst1ce s0lst1ce merged commit bb056a7 into master Jun 28, 2019
@s0lst1ce s0lst1ce deleted the config branch June 29, 2019 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@s0lst1ce s0lst1ce

Labels
bug Something isn't working enhancement New feature or request quality all issues which only affect the way code is written and that won't affect functionnality
Projects
None yet
Milestone
v1
Development

Successfully merging this pull request may close these issues.
1 participant
@s0lst1ce