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

Feat [ notifier ]: new notification pattern #326

Open
wants to merge 19 commits into
base: master
Choose a base branch
from
Open

Feat [ notifier ]: new notification pattern #326

Show file tree
Hide file tree
Changes from 9 commits
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
6281477
add redis helper for notifier worker
e11sy Dec 8, 2024
6a06087
add notifier methods for threshold evaluation
e11sy Dec 8, 2024
ced3bef
clean up in notifier
e11sy Dec 8, 2024
a348328
get rid of the buffer
e11sy Dec 8, 2024
a77b393
fix lint issues
e11sy Dec 8, 2024
e6b521b
refactor notifier and tests
e11sy Dec 18, 2024
5c4d926
refactor notifier and tests
e11sy Dec 18, 2024
2f3661a
lint fix and remove redundant
e11sy Dec 18, 2024
3184566
remove unneeded database connection
e11sy Dec 18, 2024
a740acf
improve naming and lua logic
e11sy Dec 18, 2024
63447b2
implement redis testcontainer
e11sy Dec 21, 2024
0a66ae0
refactor notifier tests with testcontainer
e11sy Dec 21, 2024
cf9545f
lint fix
e11sy Dec 21, 2024
0c59378
typo fix
e11sy Dec 21, 2024
81e808b
rebase from master
e11sy Dec 21, 2024
fb62731
update lock file
e11sy Dec 21, 2024
9f6b401
fix lua script and naming
e11sy Dec 21, 2024
f7fed46
move Date.now() to variables
e11sy Dec 21, 2024
d50efc6
update from main
e11sy Dec 25, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,9 @@
"typescript": "^3.8.3",
"uuid": "^8.3.0",
"winston": "^3.2.1",
"yup": "^0.28.5"
"yup": "^0.28.5",
"@types/redis": "^2.8.28",
"redis": "^4.7.0"
},
"devDependencies": {
"@shelf/jest-mongodb": "^1.2.3",
Expand All @@ -73,7 +75,6 @@
"jest": "25.5.4",
"nodemon": "^2.0.3",
"random-words": "^1.1.1",
"redis-mock": "^0.56.3",
"ts-jest": "25.4.0",
"wait-for-expect": "^3.0.2",
"webpack": "^4.43.0",
Expand Down
2 changes: 2 additions & 0 deletions workers/notifier/.env.sample
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
# Url for connecting to Redis
REDIS_URL=redis://redis:6379
6 changes: 5 additions & 1 deletion workers/notifier/package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,5 +3,9 @@
"version": "1.0.0",
"main": "src/index.ts",
"license": "MIT",
"workerType": "notifier"
"workerType": "notifier",
"dependencies": {
"@types/redis": "^2.8.28",
"redis": "^4.7.0"
}
}
270 changes: 0 additions & 270 deletions workers/notifier/src/buffer.ts
View file
Open in desktop

This file was deleted.

Loading
Loading