-
Notifications
You must be signed in to change notification settings - Fork 88
feat: adds factory pattern for relay class, to support future redis client … #4471
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
Merged
konstantinabl
merged 31 commits into
main
from
4470-refactor-relay-initialization-to-use-static-factory-method-pattern
Oct 20, 2025
+833
−671
Merged
Changes from 2 commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
e75f585
adds factory pattern for relay class, to support future redis client …
konstantinabl 7d14b31
imports from dist
konstantinabl 84a514c
fixes acceptance tests
konstantinabl 3e06df5
fixes integration tests
konstantinabl 1671238
fix relay logger
konstantinabl 8ad1fdd
remove relay return
konstantinabl 930a838
adds register clean
konstantinabl 43ca2a8
fixes ws server tests and transforms them to unit
konstantinabl 89916d5
removes eslint disable
konstantinabl 782b71a
addresses PR comments
konstantinabl 51cce80
fixes tests in server.spec
konstantinabl dd8477e
removes eslint disabled
konstantinabl a8ee86e
fixes comments
konstantinabl 0063ff1
improves commenting
konstantinabl ffd9d08
removes unused variables
konstantinabl bd2f3bb
reverts check out of scope
konstantinabl 09dad39
removes require from acceptance tests
konstantinabl d59e2d7
removes unused variable
konstantinabl 96e560c
removes unused import
konstantinabl e7ea819
adds dynamic loading for tier variables in method config
konstantinabl dce661d
fixes relay init
konstantinabl 002ddbb
fixes unit tests
konstantinabl 894988c
turnoff unused expression rules for tests
konstantinabl 5412778
removes unused var app
konstantinabl 1af001d
fixes ws server tests
konstantinabl 409cdeb
reverts changes to assertions
konstantinabl 67f6e89
removes unused eslint disable
konstantinabl 5138a52
improves integration test
konstantinabl 3d7d9cd
removes all instantiations using new isntead of init from tests
konstantinabl dd4e210
removes unused methods
konstantinabl 071aa7c
adds comment
konstantinabl 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
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or 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 hidden or 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 hidden or 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.
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.
Uh oh!
There was an error while loading. Please reload this page.