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

The Great Schism #6

Draft
wants to merge 53 commits into
base: main
Choose a base branch
from
Draft

The Great Schism #6

wants to merge 53 commits into from

Conversation

denverquane
Copy link
Contributor

@denverquane denverquane commented Jan 5, 2021 •
edited
Loading

Refactors Galactus to

  1. Not handle connections from capture clients; these connections will instead be handled by automuteus/wingman
  2. Handle all inbound/outbound Discord requests to help reduce complexity in the core bot business logic of automuteus
  3. Utilize a proper shard manager within the process; prevent scaling 20+instances of the bot, and instead having 2-3 instances of Galactus with 10+ shards each
  4. Implement more rigorous and structured Logging
  5. Include unit tests and mocks

@denverquane denverquane mentioned this pull request Jan 11, 2021
Closed
CarbonNeuron
CarbonNeuron suggested changes Jan 20, 2021
View reviewed changes
Copy link

@CarbonNeuron CarbonNeuron left a comment

Choose a reason for hiding this comment

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

I do agree, The event payload has been updated.
Make sure to add easter eggs, easter is soon.

@automuteus automuteus deleted a comment from CarbonNeuron Feb 2, 2021
denverquane and others added 12 commits February 1, 2021 21:41
@denverquane
Change locking strategy to explicitly not unlock (no duplicate proces…
66ca6e4 
…sing)
@denverquane
Added Premium endpoint to server and client code, alongside caching v…
27ac0dd 
…ia Redis
@CarbonNeuron
Add Content-Type header to stats endpoint
10e59f7 
This fixes #7
@denverquane
Merge pull request #9 from CarbonNeuron/patch-1
2652b03 
Add Content-Type header to stats endpoint
@denverquane
Misc ratelimit and message fixes
be6f14c
@denverquane
Merge remote-tracking branch 'origin/theGreatSchism' into theGreatSchism
2b10b22
@denverquane
Embed validation on empty fields/names
1f34933
@denverquane
Capture endless broadcast fix
1bd854d
@denverquane
Don't cache self-host values
1503b9a
@kurokobo
fix(shard_manager): #10 Move AddHandlers before start()
301c081
@denverquane
Merge pull request #11 from kurokobo/#10_shard
9cb4173 
fix(shard_manager): #10 Move AddHandlers before start()
@denverquane
Small fixes to test files
bbc4206
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@CarbonNeuron CarbonNeuron CarbonNeuron approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@denverquane @CarbonNeuron @kurokobo