-
Notifications
You must be signed in to change notification settings - Fork 59
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
Refactor RPC and rewrite aepp-rpc to TypeScript #1542
Commits on May 27, 2022
-
refactor(AeppRpc)!: make to init in sync
BREAKING CHANGE: AeppRpc doesn't accept `connection` anymore Use `connectToWallet` method instead.
Configuration menu - View commit details
-
Copy full SHA for 86c9d6c - Browse repository at this point
Copy the full SHA 86c9d6cView commit details -
Configuration menu - View commit details
-
Copy full SHA for de4c21d - Browse repository at this point
Copy the full SHA de4c21dView commit details -
Configuration menu - View commit details
-
Copy full SHA for e2b5186 - Browse repository at this point
Copy the full SHA e2b5186View commit details -
refactor(rpc-client)!: provide method handlers instead of onMessage
BREAKING CHANGE: `handlers` parameter is removed in RpcClient Provide a `methods` parameter instead of `handlers[0]`. Provide an `onDisconnect` parameter instead of `handlers[1]`.
Configuration menu - View commit details
-
Copy full SHA for 5f1a007 - Browse repository at this point
Copy the full SHA 5f1a007View commit details -
Configuration menu - View commit details
-
Copy full SHA for c8151ff - Browse repository at this point
Copy the full SHA c8151ffView commit details
Commits on May 28, 2022
-
Configuration menu - View commit details
-
Copy full SHA for 4b41b89 - Browse repository at this point
Copy the full SHA 4b41b89View commit details -
refactor(rpc-client)!: add notify method
BREAKING CHANGE: `sendMessage` of RpcClient is a private method Use `request` or `notify` instead. BREAKING CHANGE: `shareWalletInfo` of WalletRpc accepts rpcClientId instead of callback For example, rewrite ``` const connection = new BrowserRuntimeConnection({ port }) aeSdk.addRpcClient(connection) aeSdk.shareWalletInfo(port.postMessage.bind(port)) ``` to ``` const connection = new BrowserRuntimeConnection({ port }) const rpcClientId = aeSdk.addRpcClient(connection) aeSdk.shareWalletInfo(rpcClientId) ```
Configuration menu - View commit details
-
Copy full SHA for 9e97a1a - Browse repository at this point
Copy the full SHA 9e97a1aView commit details
Commits on May 29, 2022
-
refactor(aepp-rpc): depend on simplified version of RpcClient
BREAKING CHANGE: `connectToWallet` accepts wallet connection as the first argument See connect-aepp-to-wallet.md for details. BREAKING CHANGE: `disconnectWallet` runs in sync and `sendDisconnect` arg removed So, aepp would always send `closeConnection` notification. BREAKING CHANGE: `sendConnectRequest` removed Use `connectToWallet` instead.
Configuration menu - View commit details
-
Copy full SHA for f329549 - Browse repository at this point
Copy the full SHA f329549View commit details -
refactor: drop DuplicateCallbackError error
Message counted is maintained locally, so this shouldn't be a case
Configuration menu - View commit details
-
Copy full SHA for b3a2af6 - Browse repository at this point
Copy the full SHA b3a2af6View commit details -
Configuration menu - View commit details
-
Copy full SHA for b8f76ca - Browse repository at this point
Copy the full SHA b8f76caView commit details -
Configuration menu - View commit details
-
Copy full SHA for 1a8e830 - Browse repository at this point
Copy the full SHA 1a8e830View commit details