Skip to content

Conversation

@dmitrylavrenov
Copy link
Contributor

@dmitrylavrenov dmitrylavrenov commented Nov 23, 2022

@dmitrylavrenov dmitrylavrenov marked this pull request as ready for review November 23, 2022 16:31
@MOZGIII
Copy link
Contributor

MOZGIII commented Nov 23, 2022

I've completed my initial review and left some comments. There are some changes that don't seem to be necessary, and some changes that I've noted I'd like to discuss. I'll take a deeper look into the codebase of the substrate to assert that we have included all the changes.

In the meantime, can we switch to using our own branch locked/... at frontier too?

Copy link
Contributor

@MOZGIII MOZGIII left a comment

Choose a reason for hiding this comment

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

Looks good, except we must create a couple of issues to review the new parameters.

@dmitrylavrenov
Copy link
Contributor Author

Looks good, except we must create a couple of issues to review the new parameters.

Created.

@dmitrylavrenov dmitrylavrenov merged commit a1b16a5 into master Nov 23, 2022
@dmitrylavrenov dmitrylavrenov deleted the polkadot-v0.9.32 branch November 23, 2022 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants