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

Update openbrush-follow-up-2.md #1237

Merged
merged 1 commit into from
Oct 26, 2022
Merged

Conversation

coreggon11
Copy link
Contributor

@coreggon11 coreggon11 commented Oct 26, 2022

Project Abstract

This is an update of an already approved grant.
Supercolony as you know it will cease to exist in a few weeks.
The sleeping partner that Markian started Supercolony with no longer shares the vision and mission of our company. Mainly because of a different values mindset. The best option is to separate at this point.
and we would like to continue infrastructure development and enhancing Polkadot with a new identity now - 727.ventures.

Many developers are asking for the feature to test cross-contract calls in unit tests because it is hard to mock it each time and makes the testing much harder, forcing people to write integration tests with Redspot, Swanky, and Typechain-Polkadot.

We want to prioritize this task and deliver the trait refactoring in ink! later(Initially, we plan to implement an advanced testing framework in milestone eight it is why we can swap those tasks). Because it is a hot topic and after discussion in the issue and element channel, it is clear how it should be implemented=)

The scope of work is the same and also related to the ink! repository. Because it requires modification of the off-chain environment.

Grant level

  • Level 1: Up to $10,000, 2 approvals
  • Level 2: Up to $30,000, 3 approvals
  • Level 3: Unlimited, 5 approvals (for >$100k: Web3 Foundation Council approval)

#921

@Noc2 Noc2 self-assigned this Oct 26, 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.

Thanks for the update. Could @xgreenx and @TtomaS7 confirm here that they are fine with these changes? They are the two contributors of the initial PR, see https://github.com/w3f/Grants-Program/blob/master/applications/openbrush-follow-up-2.md.

@Noc2 Noc2 added the changes requested The team needs to clarify a few things first. label Oct 26, 2022
@TtomaS7
Copy link
Contributor

TtomaS7 commented Oct 26, 2022

hello!
@Noc2 yes, I confirm!

@xgreenx
Copy link
Contributor

xgreenx commented Oct 26, 2022

And I confirm!=)

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.

Perfect. Thanks. I will mark the amendment as ready for review and approve it myself.

@Noc2 Noc2 added ready for review The project is ready to be reviewed by the committee members. and removed changes requested The team needs to clarify a few things first. labels Oct 26, 2022
@takahser takahser merged commit 206d7df into w3f:master Oct 26, 2022
@coreggon11 coreggon11 mentioned this pull request Dec 14, 2022
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

8 participants