Skip to content

feat(tendermint): implement better sequence resolving logic#2164

Merged
onur-ozkan merged 5 commits intodevfrom
tendermint-nucleus-sequence
Jul 17, 2024
Merged

feat(tendermint): implement better sequence resolving logic#2164
onur-ozkan merged 5 commits intodevfrom
tendermint-nucleus-sequence

Conversation

@onur-ozkan
Copy link
Copy Markdown

This change makes kdf to resolve account sequence numbers locally without needing to perform RPC requests. In addition, avoids the use of incorrect sequence numbers that may come from cached responses.

Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
@onur-ozkan onur-ozkan force-pushed the tendermint-nucleus-sequence branch from 95f87ef to aa09764 Compare July 15, 2024 14:09
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
@gcharang gcharang self-requested a review July 17, 2024 07:44
Copy link
Copy Markdown

@gcharang gcharang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tested via GUI, works!

Copy link
Copy Markdown
Collaborator

@mariocynicys mariocynicys left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@onur-ozkan onur-ozkan merged commit 4db6c66 into dev Jul 17, 2024
@onur-ozkan onur-ozkan deleted the tendermint-nucleus-sequence branch July 17, 2024 07:47
shamardy added a commit that referenced this pull request Jul 18, 2024
dimxy pushed a commit that referenced this pull request Jul 21, 2024
* dev:
  feat(nft-swap): add standalone maker contract and proxy support (#2100)
  feat(ETH): add `gas_limit` coins param to override default values (#2137)
  feat(tendermint): implement better sequence resolving logic (#2164)
  ci(artifact): add target for macos on apple silicon (#2163)
  fix(helpers): extend http to ws address conversion (#2166)
  fix(makerbot): add "testcoin" to provider options (#2161)
  fix(hd_wallet): make extended pubkey of hd wallet generic (#2159)
  fix(docker-tests): implement containers runtime directories (#2162)
  feat(tendermint): improve the `max` handling for tendermint withdraw (#2155)
  revert #2158 (comment) (#2160)
  ci(artifacts): upload build artifacts with in-tree script (#2158)
  test(tendermint): migrate to local/offline containerized testnets (#2128)
  use easingthemes/ssh-deploy@v5.0.3 for all builds except windows (#2157)
  chore(bin): rename mm2 binaries to kdf (#2126)
dimxy pushed a commit to dimxy/komodo-defi-framework that referenced this pull request Aug 12, 2024
* dev: (22 commits)
  chore(release): bump mm2 version to 2.2.0-beta (GLEECBTC#2188)
  ci(docker-tests): ignore tendermint IBC tests for now (GLEECBTC#2185)
  feat(nft-swap): complete refund methods (GLEECBTC#2129)
  chore(release): add changelog entries for v2.1.0-beta (GLEECBTC#2165)
  fix(zcoin): don't force low r signing to generate htlc pubkey for zcoin (GLEECBTC#2184)
  chore(rust-analyzer): add rust-analyzer into the workspace toolchain (GLEECBTC#2179)
  chore: migrate .cargo/config to .cargo/config.toml to avoid deprecation warning (GLEECBTC#2177)
  fix(swaps): ensure taker payment spend confirmations (GLEECBTC#2176)
  feat(nft-swap): add standalone maker contract and proxy support (GLEECBTC#2100)
  feat(ETH): add `gas_limit` coins param to override default values (GLEECBTC#2137)
  feat(tendermint): implement better sequence resolving logic (GLEECBTC#2164)
  ci(artifact): add target for macos on apple silicon (GLEECBTC#2163)
  fix(helpers): extend http to ws address conversion (GLEECBTC#2166)
  fix(makerbot): add "testcoin" to provider options (GLEECBTC#2161)
  fix(hd_wallet): make extended pubkey of hd wallet generic (GLEECBTC#2159)
  fix(docker-tests): implement containers runtime directories (GLEECBTC#2162)
  feat(tendermint): improve the `max` handling for tendermint withdraw (GLEECBTC#2155)
  revert GLEECBTC#2158 (comment) (GLEECBTC#2160)
  ci(artifacts): upload build artifacts with in-tree script (GLEECBTC#2158)
  test(tendermint): migrate to local/offline containerized testnets (GLEECBTC#2128)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants