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

Update the event queue wrapper - Closes #7301 #7358

Merged
merged 4 commits into from
Aug 3, 2022

Conversation

mosmartin
Copy link
Contributor

@mosmartin mosmartin commented Aug 1, 2022

What was the problem?

This PR resolves #7301

How was it solved?

Added the create/restore snapshot functionality to the wrapper.

How was it tested?

Added and updated existing unit tests.

@mosmartin mosmartin requested a review from ishantiw August 1, 2022 12:07
@mosmartin mosmartin self-assigned this Aug 1, 2022
@ishantiw ishantiw requested a review from shuse2 August 1, 2022 12:25
Add a new method `getChildQueue ()` to replace the `wrapEventQueue()`. This refactoring also updates all instances where the `wrapEventQueue()` is used.
Add and update existing tests.
Copy link
Contributor

@ishantiw ishantiw left a comment

Choose a reason for hiding this comment

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

LGTM 🙂

Copy link
Collaborator

@shuse2 shuse2 left a comment

Choose a reason for hiding this comment

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

🚀

@shuse2 shuse2 merged commit ef942e7 into development Aug 3, 2022
@shuse2 shuse2 deleted the 7301-update-event-queue-wrapper branch August 3, 2022 07:09
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.

Update eventQueue to give a wrapper to manage events at crossChainCommand level
3 participants