-
Notifications
You must be signed in to change notification settings - Fork 41
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
fix: remote peer rejected #1645
Conversation
…/remote-peer-rejected
…/remote-peer-rejected
size-limit report 📦
|
To be merged only when node tests will run on a nwaku version that contains this fix |
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.
looks great! thanks @fbarbu15
perhaps we should coordinate with @waku-org/nwaku-developers to check when the next release is happening, and move this to draft until then.
@danisharora099 the next release is currently scheduled for 2023-10-18: https://github.com/waku-org/nwaku/milestone/13 @gabrielmer and @vpavlin will prepare the release items. |
@@ -19,7 +19,7 @@ import { | |||
TestEncoder | |||
} from "./utils.js"; | |||
|
|||
describe("Waku Light Push", function () { | |||
describe.only("Waku Light Push", function () { |
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.
@fbarbu15 !!!
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.
Thanks, will keep an eye out for this in the future
This one was removed imediately after with this commit
Problem
Failing light push tests were returning wrong error message
Solution
Since waku-org/nwaku#1641 was fixed we need to correct the error message in the tests