ArkProjectNFTs / ark-project

ArkProject is a liquidity layer for digital assets, uniting markets, empowering creators, and bridging the gap to mass adoption. Built on top of Starknet, ArkProject is designed to provide a fully decentralized and trustless orderbook on-chain.
https://arkproject.dev
Apache License 2.0
27 stars 26 forks source link

fix(diri): ensure events are handled sorted by block number #425

Open ptisserand opened 3 months ago

ptisserand commented 3 months ago

Description

Add block number sorting to ensure that events handling respect temporal order.

What type of PR is this? (check all applicable)

Related Tickets & Documents

Added tests?

Added to documentation?

[optional] Are there any post-deployment tasks we need to perform?

[optional] What gif best describes this PR or how it makes you feel?

PR Title and Description Guidelines:

Closing Issues

linear[bot] commented 3 months ago

DEV-714 Cancelled offer event not inserted by Arkchain indexer

changeset-bot[bot] commented 3 months ago

āš ļø No Changeset found

Latest commit: 2cf652ae823a9344fbe42f4f1c4e83cc443f1cb7

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

vercel[bot] commented 3 months ago

The latest updates on your projects. Learn more about Vercel for Git ā†—ļøŽ

Name Status Preview Comments Updated (UTC)
ark-project-sdk-demo āœ… Ready (Inspect) Visit Preview šŸ’¬ Add feedback Aug 23, 2024 7:51am