Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change the scope of work of the 5th milestone in OpenBrush #848

Merged
merged 1 commit into from
Mar 1, 2022

Conversation

xgreenx
Copy link
Contributor

@xgreenx xgreenx commented Mar 1, 2022

We want to change the scope of work of the 5th milestone in OpenBrush.
We've changed point 2: "Add support of default implementation in trait definition on ink! level" -> "Re-work the storage of contracts"

We consider that reworking of the contract's storage should be done before default implementation in traits and it is more important. We participated in the discussion and we have a clear vision of how to implement that.

That change requires the same human resources as default implementation in traits so the cost of the milestone is the same.
We moved default implementation in traits to the third Grant.

@alxs alxs self-assigned this Mar 1, 2022
Copy link
Contributor

@alxs alxs left a comment

Choose a reason for hiding this comment

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

Thanks for keeping us in the loop @xgreenx. Sounds good.

@alxs alxs added amendment This PR proposes changes to an existing application. ready for review The project is ready to be reviewed by the committee members. labels Mar 1, 2022
Copy link
Collaborator

@Noc2 Noc2 left a comment

Choose a reason for hiding this comment

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

Sound good and thanks for the update

Copy link
Collaborator

@takahser takahser left a comment

Choose a reason for hiding this comment

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

Sounds good to me as well. Thanks guys! 👍

@Noc2 Noc2 merged commit 9c2d01a into w3f:master Mar 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
amendment This PR proposes changes to an existing application. ready for review The project is ready to be reviewed by the committee members.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants