Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix(store-sync): reduce latency in waitForTransaction #2665

Merged
merged 28 commits into from
Apr 16, 2024
Merged

fix(store-sync): reduce latency in waitForTransaction #2665

merged 28 commits into from
Apr 16, 2024

Conversation

alvrs
Copy link
Member

@alvrs alvrs commented Apr 15, 2024

No description provided.

alvrs and others added 24 commits April 12, 2024 01:43
@alvrs
feat(common): send fetch requests in parallel
bc4be63
@alvrs
feat(common): do not dynamically fetch fixed tx params
527eecc
@alvrs
fix: bring back separate gas estimation
3e9ef7f
@alvrs
feat(common): refresh fees in intervals instead of right before every…
814ae1e 
… transaction
@alvrs
feat(common): don't care about max fee during gas estimation
58a0464
@alvrs
remove stubbed values from result
7b5d7ef
@alvrs
fix: gas estimation
858f163
@alvrs
update viem
2ecc4d9
@alvrs
self-review
260d791
@alvrs
Merge branch 'main' into latency
702eccb
@alvrs
run all-install
f3daaf9
@alvrs
don't throw
a592fa5
@holic
bump viem to 2.9.16
b31383f
@alvrs
fix nonce handling
cec07e8
@alvrs
use getAction
d62171a
@alvrs
bring back block tag
1e8acce
@holic
cast return type for now
b366ebf
@alvrs
Merge branch 'holic/viem-2.9' into latency
24610a1
@alvrs
update log
7e6635e
@alvrs
Merge branch 'main' into latency
fc7824c
@alvrs
Create fresh-nails-draw.md
bbc1cee
@alvrs
review feedback
c61f8dd
@alvrs
Merge branch 'main' into latency
263c679
@alvrs
pull out store-sync changes
99d7a29
@alvrs alvrs requested review from holic and yonadaa as code owners April 15, 2024 18:53
@changeset-bot changeset-bot
Copy link

changeset-bot bot commented Apr 15, 2024 •
edited
Loading

🦋 Changeset detected

Latest commit: 615664a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 24 packages
Name Type
@latticexyz/store-sync Patch
@latticexyz/dev-tools Patch
@latticexyz/store-indexer Patch
@latticexyz/abi-ts Patch
@latticexyz/block-logs-stream Patch
@latticexyz/cli Patch
@latticexyz/common Patch
@latticexyz/config Patch
create-mud Patch
@latticexyz/faucet Patch
@latticexyz/gas-report Patch
@latticexyz/protocol-parser Patch
@latticexyz/query Patch
@latticexyz/react Patch
@latticexyz/recs Patch
@latticexyz/schema-type Patch
@latticexyz/services Patch
solhint-config-mud Patch
solhint-plugin-mud Patch
@latticexyz/store Patch
@latticexyz/utils Patch
@latticexyz/world-modules Patch
@latticexyz/world Patch
mock-game-contracts Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@alvrs alvrs mentioned this pull request Apr 15, 2024
Merged
holic
holic previously approved these changes Apr 15, 2024
View reviewed changes
Copy link
Member

@holic holic left a comment

Choose a reason for hiding this comment

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

🙏

Base automatically changed from latency to main April 16, 2024 13:14
@alvrs alvrs dismissed holic’s stale review April 16, 2024 13:14

The base branch was changed.

@alvrs alvrs merged commit de3bc3d into main Apr 16, 2024
11 of 12 checks passed
@alvrs alvrs deleted the latency-2 branch April 16, 2024 13:15
@github-actions github-actions bot mentioned this pull request Apr 22, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@holic holic holic left review comments

@yonadaa yonadaa Awaiting requested review from yonadaa

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@alvrs @holic