-
Notifications
You must be signed in to change notification settings - Fork 522
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
#trivial Clean up cross domain communicator #1253
Conversation
packages/wallet-sdk/src/core/communicator/CrossDomainCommunicator.ts
Outdated
Show resolved
Hide resolved
This reverts commit 5669680. # Conflicts: # packages/wallet-sdk/src/core/communicator/PopUpCommunicator.ts
7a10359
to
d8cd911
Compare
3bad89e
to
f84240a
Compare
Pull request was converted to draft
commit 8067edd Author: Jungho Bang <jungho.bang@coinbase.com> Date: Wed May 1 01:04:15 2024 -0700 min diff commit 0fd1b72 Author: Jungho Bang <jungho.bang@coinbase.com> Date: Wed May 1 01:00:27 2024 -0700 fix test commit 487870a Author: Jungho Bang <jungho.bang@coinbase.com> Date: Wed May 1 00:52:53 2024 -0700 cleaner commit b427e5a Author: Jungho Bang <jungho.bang@coinbase.com> Date: Wed May 1 00:46:41 2024 -0700 hmmm. back to abstract commit ba9c7b0 Author: Jungho Bang <jungho.bang@coinbase.com> Date: Wed May 1 00:27:13 2024 -0700 SignHandler is PopUpCommunicator commit 86e585a Merge: 1e1fb4c 07a99c5 Author: Jungho Bang <jungho.bang@coinbase.com> Date: Tue Apr 30 23:43:18 2024 -0700 Merge branch 'jungho/cleanup-xdomain-comm' into jungho/refactor-puc # Conflicts: # packages/wallet-sdk/src/core/communicator/PopUpCommunicator.ts commit 1e1fb4c Author: Jungho Bang <jungho.bang@coinbase.com> Date: Tue Apr 30 23:40:05 2024 -0700 cleaner # Conflicts: # packages/wallet-sdk/src/core/communicator/PopUpCommunicator.ts commit ac2e71d Merge: d6c95a6 16f01bb Author: Jungho Bang <jungho.bang@coinbase.com> Date: Tue Apr 30 23:30:23 2024 -0700 Merge branch 'jungho/cleanup-xdomain-comm' into jungho/refactor-puc commit d6c95a6 Author: Jungho Bang <jungho.bang@coinbase.com> Date: Tue Apr 30 23:28:11 2024 -0700 no temp vars commit ffaac3c Author: Jungho Bang <jungho.bang@coinbase.com> Date: Tue Apr 30 23:23:11 2024 -0700 min diff commit bfb1cf7 Merge: 934757c f84240a Author: Jungho Bang <jungho.bang@coinbase.com> Date: Tue Apr 30 23:19:54 2024 -0700 Merge branch 'jungho/cleanup-xdomain-comm' into jungho/refactor-puc commit 934757c Author: Jungho Bang <jungho.bang@coinbase.com> Date: Tue Apr 30 23:01:07 2024 -0700 WIP # Conflicts: # packages/wallet-sdk/src/core/communicator/CrossDomainCommunicator.ts commit 39bacbe Author: Jungho Bang <jungho.bang@coinbase.com> Date: Tue Apr 30 23:07:10 2024 -0700 Revert "revert onConnect" This reverts commit d8cd911.
protected abstract setupPeerWindow(): Promise<void>; | ||
// returns true if the message is handled | ||
protected abstract handleIncomingMessage(_: Message): Promise<boolean>; |
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.
will use this boolean value in the followup pr
fyi @fan-zhang-sv i am closing this. |
this.connected = true; | ||
} | ||
|
||
disconnect(): void { |
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.
FE doesn't use onDisconnect nor disconnect
Review Error for fan-zhang-sv @ 2024-05-01 20:17:55 UTC |
Summary
How did you test your changes?