Skip to content

Add EventEmitter to XCM Executor#7234

Merged
acatangiu merged 90 commits intoparitytech:masterfrom
jpserrat:jpserrat/add-sent-event
Mar 10, 2025
Merged

Add EventEmitter to XCM Executor#7234
acatangiu merged 90 commits intoparitytech:masterfrom
jpserrat:jpserrat/add-sent-event

Conversation

@jpserrat
Copy link
Copy Markdown
Contributor

@jpserrat jpserrat commented Jan 18, 2025

Closes #6851

This PR adds an EventEmitter trait to the XCM Executor configuration, enabling event emission for XCM handling. The implementation introduces three dedicated functions to emit relevant events:

  • emit_sent_event: Emits a Sent event when an XCM is successfully sent.
  • emit_send_failure_event: Emits a SendFailed event when an XCM fails to send.
  • emit_process_failure_event: Emits a ProcessXcmError event when an XCM fails during processing.

Kusama address: FkB6QEo8VnV3oifugNj5NeVG3Mvq1zFbrUu4P5YwRoe5mQN

@jpserrat jpserrat requested a review from a team as a code owner January 18, 2025 14:05
@paritytech-review-bot paritytech-review-bot bot requested a review from a team January 18, 2025 14:06
@franciscoaguirre franciscoaguirre added the T6-XCM This PR/Issue is related to XCM. label Jan 28, 2025
@franciscoaguirre franciscoaguirre self-assigned this Jan 28, 2025
@acatangiu
Copy link
Copy Markdown
Contributor

@jpserrat do you plan on finishing this PR or should we take over?

@jpserrat
Copy link
Copy Markdown
Contributor Author

@acatangiu Sorry for the delay on this. I'm going to finish.

@raymondkfcheung
Copy link
Copy Markdown
Contributor

If #7200 is merged before your PR, you may need add the new EventEmitter on the XcmTestConfig.

@paritytech-review-bot paritytech-review-bot bot requested a review from a team February 12, 2025 21:20
@acatangiu acatangiu added this pull request to the merge queue Mar 10, 2025
@raymondkfcheung raymondkfcheung moved this from In-Review to Done in Bridges + XCM Mar 10, 2025
Merged via the queue into paritytech:master with commit ea8783b Mar 10, 2025
238 of 253 checks passed
@acatangiu acatangiu mentioned this pull request Mar 11, 2025
@franciscoaguirre franciscoaguirre removed their assignment Jun 5, 2025
@acatangiu acatangiu moved this from To be released (SDK) to SDK Released - Needs Integration in fellowship/runtimes integrations queue Jul 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T6-XCM This PR/Issue is related to XCM.

Projects

Status: Done
Status: Done
Status: SDK Released - Needs Integration

Development

Successfully merging this pull request may close these issues.

Add "Sent" event when a message is sent via execute

10 participants