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

Update mainchain registration command on sidechain module - Closes #7441 #7533

Merged
merged 1 commit into from
Sep 22, 2022

Conversation

Incede
Copy link
Contributor

@Incede Incede commented Sep 15, 2022

What was the problem?

This PR resolves #7441

How was it solved?

Update as specified

How was it tested?

Added and updated unit tests

@Incede Incede self-assigned this Sep 15, 2022
@Incede Incede marked this pull request as ready for review September 16, 2022 21:32
@Incede Incede requested a review from shuse2 September 16, 2022 21:33
@shuse2 shuse2 changed the base branch from development to 7440_sidechain_reg_update September 19, 2022 08:49
Base automatically changed from 7440_sidechain_reg_update to development September 20, 2022 16:46
@Incede Incede changed the base branch from development to feature/7211-update-modules-events-err September 21, 2022 10:06
@Incede Incede changed the base branch from feature/7211-update-modules-events-err to development September 21, 2022 10:07
@Incede Incede requested a review from shuse2 September 21, 2022 13:11
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, just a comment regarding slight variation from the LIP

@ishantiw ishantiw merged commit b2ed342 into development Sep 22, 2022
@ishantiw ishantiw deleted the 7441-mainchain-reg branch September 22, 2022 12:21
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 mainchain registration command on sidechain module
3 participants