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 CESS.md #754

Closed
wants to merge 2 commits into from
Closed

Update CESS.md #754

wants to merge 2 commits into from

Conversation

swowk
Copy link
Contributor

@swowk swowk commented Dec 28, 2021

Project Abstract

Updated the content in milestone 2, that is, replaced the CESS Contract of the storage mining module with Substrate Pallet. The reasons are as follows:

  1. In view of the fact that the file processing module and the miner registration module in the project are implemented by pallet as shown in milestone 1, and the storage mining also needs to exchange information with the other two modules. Therefore, naturally we think that using pallet to build the storage mining module will make the project better.
  2. From a design perspective, as a project of storage blockchain, the storage mining module should be the core part of the bottom layer. Therefore, implementing this as a pallet will be a better choice than as a smart contract.

For which grant level are you applying?

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

Application Checklist

  • The application template has been copied, renamed ( project_name.md) and updated.
  • A BTC or Ethereum (DAI/USDT) address for the payment of the milestones is provided inside the application.
  • I have read and acknowledged the terms and conditions.
  • The software delivered for this grant will be released under an open-source license specified in the application.
  • The initial PR contains only one commit (squash and force-push if needed).
  • The grant will only be announced once the first milestone has been accepted.

CumulusSystem and others added 2 commits August 28, 2021 15:38
@swowk
Copy link
Contributor Author

swowk commented Dec 28, 2021

@Noc2 Hello, please review our updated proposal. We have updated milestone 2, please let me know if you need more details or have any questions. Thanks

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 alot for creating the PR. However, could you just change the few lines in the original file instead of changing the entire file as part of your PR. This way everyone immediately sees the difference and sees what actually has changed.

@Noc2 Noc2 added amendment This PR proposes changes to an existing application. changes requested The team needs to clarify a few things first. labels Dec 28, 2021
@Noc2 Noc2 self-assigned this Dec 28, 2021
@swowk swowk closed this Dec 28, 2021
@swowk swowk mentioned this pull request Dec 28, 2021
9 tasks
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. changes requested The team needs to clarify a few things first.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants