Skip to content

Fix inconsistent timestamp for failed transactions#316

Merged
stackman27 merged 1 commit intodevelopfrom
sui/timestamp-fix
Jan 9, 2026
Merged

Fix inconsistent timestamp for failed transactions#316
stackman27 merged 1 commit intodevelopfrom
sui/timestamp-fix

Conversation

@pedrovsg
Copy link
Contributor

@pedrovsg pedrovsg commented Jan 9, 2026

Making blocktimestamp consistent with event indexer.

@pedrovsg pedrovsg requested a review from a team as a code owner January 9, 2026 01:30
@github-actions
Copy link

github-actions bot commented Jan 9, 2026

👋 pedrovsg, thanks for creating this pull request!

To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team.

Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks!

@stackman27 stackman27 merged commit d08e4ad into develop Jan 9, 2026
30 of 40 checks passed
@stackman27 stackman27 deleted the sui/timestamp-fix branch January 9, 2026 01:55
faisal-chainlink added a commit that referenced this pull request Jan 12, 2026
* Fix inconsistent timestamp for failed transactions (#316)

* feat(deployment): Curse Uncurse changeset (#310)

* feat(deployment): Curse Uncurse changeset

* add tests

* improve tests

---------

Co-authored-by: FelixFan1992 <fankejin@gmail.com>

* feat(deployment): Add changesets to configure MCMS and transfer ownership to self (#317)

* refactor mcms config into a sequence

* refactor to separate accept mcms ownership sequence

* changesets to accept ownership and configure mcms

* use []any instead of []uint8 for messageId and messageHash in synth events (#318)

* use []any instead of []uint8 for messageId and messageHash in synth events

* update transaction_index_test

* extend transaction_indexer test

---------

Co-authored-by: VSG <pedro.grilo@smartcontract.com>
Co-authored-by: Rodrigo Soares <38868277+rodrigombsoares@users.noreply.github.com>
Co-authored-by: FelixFan1992 <fankejin@gmail.com>
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.

2 participants