-
Notifications
You must be signed in to change notification settings - Fork 9.1k
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
feat(core): Implement inter-main communication for test webhooks in multi-main setup #8267
Merged
ivov
merged 24 commits into
master
from
pay-1189-implement-ipc-for-test-webhooks-in-multi-main-setup
Jan 12, 2024
Merged
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
4d745e6
Initial setup
ivov e44919c
Fix check
ivov 11a6078
Better naming
ivov 7e11074
Clean up push code
ivov 65fa543
Improve types
ivov a53ce37
Missed a spot
ivov d4f28bc
Cleanup
ivov 524df8d
Complete circuit
ivov 94bf1fa
Cleanup
ivov 38f7256
Better naming
ivov 21c2226
Expand comment
ivov 06aadc7
Cleanup
ivov 961af88
Optimize deregistration
ivov f02d2ba
Implement IPC for clearing test webhooks
ivov acba608
Fix some tests
ivov 6cb7b78
Merge master, fix conflicts
ivov 532d2a1
Cleanup
ivov ea7b462
Renaming
ivov c248357
Remove debounding
ivov bde1ecf
Sanity check on worker
ivov 9ddc310
Set up for expiration
ivov d32d039
Remove outdated comment
ivov 61637c5
Implement expiration
ivov 302103e
Add buffer
ivov File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Renaming
- Loading branch information
commit ea7b4628020cb71912ab4598a32e5503905ce0b5
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generally this file needs refactoring and cleanup, hopefully soon. Only added to it to avoid increasing the scope.