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

feat: replace TokenSent event with CrossSubnetMessageSent #223

Merged
merged 3 commits into from
May 3, 2023

Conversation

sebastiendan
Copy link
Member

@sebastiendan sebastiendan commented May 1, 2023

Description

With the messaging features of ToposCore now extracted into a new ToposMessaging contract, the current logic for the sequencer process to capture events linked to cross-subnet messages must be updated.

This PR updates the sequencer so that it consumes new CrossSubnetMessageSent events emitted by ToposCore.

Additions and Changes

  • Update ToposCore ABI
  • Replace the decoding of TokenSent events by the one of CrossSubnetMessageSent events
  • Update the decoding to solely decode the targetSubnetId field (other fields weren't used beforehand anyway)

PR Checklist:

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added or updated tests that comprehensively prove my change is effective or that my feature works

@sebastiendan sebastiendan self-assigned this May 1, 2023
@sebastiendan sebastiendan marked this pull request as ready for review May 1, 2023 23:06
@sebastiendan sebastiendan requested a review from a team as a code owner May 1, 2023 23:06
@sebastiendan sebastiendan requested review from gruberb and atanmarko May 1, 2023 23:06
@sebastiendan
Copy link
Member Author

@topos-network/protocol This is a suggestion of update, I will let you confirm whether it's enough or whether we want a better version of it

@sebastiendan sebastiendan merged commit df8d0ec into main May 3, 2023
@sebastiendan sebastiendan deleted the feat/use-generic-cross-subnet-msg-events branch May 3, 2023 13:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants