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

feat: redis #42

Merged
merged 30 commits into from
Dec 16, 2022
Merged

feat: redis #42

merged 30 commits into from
Dec 16, 2022

Conversation

Geczy
Copy link
Member

@Geczy Geczy commented Dec 16, 2022 •
edited

Pull Request

Linked GH Issues

closes #38

Summary of Change

  • Convert to docker
  • Convert to Redis
  • No more saving socket ids

Current Behavior

  • Using global variables

New Behavior

Using Redis JSON server

Breaking Changes and Impact/Migration

YES

Tested With

just test
albeit there are a bunch of linter warnings

Other Information

Geczy and others added 30 commits December 12, 2022 22:20
@Geczy
redis: and so we begin
cbaeeea
@Geczy
working event subs / pubs
5285474
@Geczy
rename to just .gsi
45d9ff8
@Geczy
removing gsiclient where not needed
c395d51
@Geczy
we dont need to track socket ids anymore!
81a8d66
@Geczy
fix with redis, mmr updates properly and tells chat now
2694663
@Geczy
fix for oldmmr, just needed to rearrange it
02ddc50
@Geczy
fix gsi not sending new states
3db491c
@Geczy
working better to not check name
643a0d5
@Geczy
mv twitch command
d18c364
@Geczy
add warning trace
d7229fb
@Geczy
small changes
dbb8a03
@Geczy
this could be the event emitter max reached issue
5af9d97
@Geczy
huh
3c70c7e
@Geczy
add newrelic
e0cbe8d
@Geczy
move to docker
95638ca
@GentikSolm
feat: docker facelift (#40)
45d01fc
@Geczy
sllight fixes
bad7544
@Geczy
limit memory of containers
e7e2719
@GentikSolm
fix: nodemon for docker container (#41)
1e67ec1
@Geczy
more limits
564b847
@Geczy
fix infinite mongo connect loop
c0a6869
@Geczy
add mongodb if we need it, dev only
42544c7
@Geczy
add steam volume
db94469
@Geczy
update gitignore & env
c35a52a
@Geczy
we dont need dotenv package since docker injects env variables from .…
1977ebb 
…env files
@Geczy
only use port 5000
6216502
@Geczy
rm newrelic, consider using docker
af22c18
@Geczy
Merge branch 'master' into feat/redis
7d5be2e
@Geczy
fix conflicts
c102825
@Geczy Geczy marked this pull request as ready for review December 16, 2022 13:22
@Geczy Geczy changed the title redis: and so we begin feat: redis Dec 16, 2022
@Geczy Geczy merged commit 2a210d1 into master Dec 16, 2022
@Geczy Geczy deleted the feat/redis branch December 16, 2022 13:23
@Geczy Geczy restored the feat/redis branch August 20, 2023 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

redis, docker
2 participants
@Geczy @GentikSolm