-
Notifications
You must be signed in to change notification settings - Fork 174
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
[Push] Sync subscriptions #915
Conversation
Sources/WalletConnectPush/Client/Common/DeletePushSubscriptionSubscriber.swift
Outdated
Show resolved
Hide resolved
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 for taking a look at it so fast 🙏
I understand that this adds sync store for threads but we also need to sync thread keys
and do the same for messages, correct?
...onnectPush/Client/Wallet/ProtocolEngine/wc_notifyUpdate/NotifyUpdateResponseSubscriber.swift
Outdated
Show resolved
Hide resolved
63c113d
to
d8b9365
Compare
3358b98
to
c8be622
Compare
yes, we're discussing keys sync here: https://github.com/WalletConnect/walletconnect-docs/pull/773/files |
5c69171
to
744af6f
Compare
910ce67
to
b972182
Compare
...nectPush/Client/Wallet/ProtocolEngine/wc_pushSubscribe/PushSubscribeResponseSubscriber.swift
Show resolved
Hide resolved
...nectPush/Client/Wallet/ProtocolEngine/wc_pushSubscribe/PushSubscribeResponseSubscriber.swift
Outdated
Show resolved
Hide resolved
...nectPush/Client/Wallet/ProtocolEngine/wc_pushSubscribe/PushSubscribeResponseSubscriber.swift
Outdated
Show resolved
Hide resolved
1f02c87
to
86c4218
Compare
7989dc1
to
75bc40d
Compare
@@ -21,10 +23,19 @@ final class PushTests: XCTestCase { | |||
|
|||
var pairingStorage: PairingStorage! | |||
|
|||
let pk = try! EthereumPrivateKey() |
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.
sometimes we re running those tests against prod cast and prod gm dapp. Can we use some fixed entropy here for deriving the private key? to not pollute their DBs
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.
Why we can pollute db's? I think there will be same amount of relay messages in both cases (random and derived private key)
Description
Resolves # (issue)
How Has This Been Tested?
Due Dilligence