Skip to content

Commit

Permalink
Fix mismatch between legacy and new transactions API (#677)
Browse files Browse the repository at this point in the history
* Fix mismatch between legacy and new transactions API

* PR link

* Add `erroneously` in CHANGELOG
  • Loading branch information
tomaka authored Jun 6, 2023
1 parent 154d52d commit abad638
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
4 changes: 2 additions & 2 deletions light-base/src/json_rpc_service/background/transactions.rs
Original file line number Diff line number Diff line change
Expand Up @@ -269,7 +269,7 @@ impl<TPlat: PlatformRef> Background<TPlat> {
};

let update = match (status_update, is_legacy) {
(transactions_service::TransactionStatus::Broadcast(peers), false) => {
(transactions_service::TransactionStatus::Broadcast(peers), true) => {
methods::ServerToClient::author_extrinsicUpdate {
subscription: (&subscription_id).into(),
result: methods::TransactionStatus::Broadcast(
Expand All @@ -278,7 +278,7 @@ impl<TPlat: PlatformRef> Background<TPlat> {
}
.to_json_call_object_parameters(None)
}
(transactions_service::TransactionStatus::Broadcast(peers), true) => {
(transactions_service::TransactionStatus::Broadcast(peers), false) => {
num_broadcasted_peers += peers.len();
methods::ServerToClient::transaction_unstable_watchEvent {
subscription: (&subscription_id).into(),
Expand Down
4 changes: 4 additions & 0 deletions wasm-node/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,10 @@

## Unreleased

### Fixed

- Fix `author_submitAndWatchExtrinsic` erroneously generating `transaction_unstable_watchEvent` notifications, and `transaction_unstable_submitAndWatch` erroneously generating `author_extrinsicUpdate` notifications. ([#677](https://github.com/smol-dot/smoldot/pull/677))

## 1.0.8 - 2023-06-05

### Changed
Expand Down

0 comments on commit abad638

Please sign in to comment.