Skip to content
This repository has been archived by the owner on Aug 2, 2024. It is now read-only.

Bug: fix failed l1 message tx replay #1513

Closed
wants to merge 1 commit into from

Conversation

prtk418
Copy link

@prtk418 prtk418 commented Mar 16, 2024

Pull Request type

Please add the labels corresponding to the type of changes your PR introduces:

  • Bugfix

Resolves: #1504

Comment on lines +693 to +697
storage::transactional::with_transaction(|| {
storage::TransactionOutcome::Commit(Result::<_, DispatchError>::Ok(
L1Messages::<T>::mutate(|nonces| nonces.insert(nonce))
))
})?;
Copy link
Author

Choose a reason for hiding this comment

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

@apoorvsadana Committing nonce data separately for error case, however it still doesn't commit to db. Can you please check once and help me out here.

@prtk418
Copy link
Author

prtk418 commented Mar 18, 2024

PR on hold because the issue might potentially get fixed by a rebase on progress by @tdelabro

Copy link

There hasn't been any activity on this pull request recently, and in order to prioritize active work, it has been marked as stale.
This PR will be closed and locked in 7 days if no further activity occurs.
Thank you for your contributions!

@github-actions github-actions bot added the stale label Apr 18, 2024
@tdelabro
Copy link
Collaborator

@prtk418 #1566 was merged, can you rebase yourself and check if the issue is still there and if so, fix it

@github-actions github-actions bot removed the stale label Apr 27, 2024
Copy link

There hasn't been any activity on this pull request recently, and in order to prioritize active work, it has been marked as stale.
This PR will be closed and locked in 7 days if no further activity occurs.
Thank you for your contributions!

@github-actions github-actions bot added the stale label May 27, 2024
@tdelabro
Copy link
Collaborator

@prtk418 are you still working on this?

@prtk418
Copy link
Author

prtk418 commented May 28, 2024

@prtk418 are you still working on this?

Hi @tdelabro , yes I will be picking this back up.

@github-actions github-actions bot removed the stale label May 29, 2024
Copy link

There hasn't been any activity on this pull request recently, and in order to prioritize active work, it has been marked as stale.
This PR will be closed and locked in 7 days if no further activity occurs.
Thank you for your contributions!

@github-actions github-actions bot added the stale label Jun 29, 2024
@tdelabro
Copy link
Collaborator

tdelabro commented Aug 2, 2024

repository archived in favor of https://github.com/madara-alliance/madara

@tdelabro tdelabro closed this Aug 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug: l1 message is processed repeatedly
2 participants