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 README.md #38

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update README.md #38

wants to merge 2 commits into from

Conversation

TtomaS7
Copy link
Contributor

@TtomaS7 TtomaS7 commented Jan 21, 2025

I've updated the description.

I've updated the description.
@TtomaS7 TtomaS7 requested a review from semuelle January 21, 2025 09:21
Copy link
Contributor

@semuelle semuelle left a comment

Choose a reason for hiding this comment

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

Thanks, Toma. I left a few comments and suggestions. Maybe someone closer to the tech can comment as well, e.g. on sol2ink.

- **Facilitator**:
- [Toma Sadova](https://www.linkedin.com/in/toma-sadova/)

The Ecosystem Grants are only one aspect of the ink!ubator. For more details on the rest of the initiative, see the [Polkadot Treasury Bounty Proposal](https://polkadot.polkassembly.io/bounty/19).
Copy link
Contributor

@semuelle semuelle Jan 21, 2025

Choose a reason for hiding this comment

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

Is this still true about it being one aspect of the ink!ubator?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

agree, we can remove it

Comment on lines +45 to -41
**Guidelines**

### Guidelines
Copy link
Contributor

Choose a reason for hiding this comment

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

If you remove the headings ("###"), the links in the table of contents won't work anymore.

The main acceptance criteria of any submitted project would be its impact on the ink! ecosystem.
---

**Project Ideas**
Copy link
Contributor

Choose a reason for hiding this comment

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

see above


**Technical Showcase**
We recommend using [Sol2ink](https://github.com/Brushfam/sol2ink) to accelerate development. Learn more in the [Sol2ink docs](https://learn.brushfam.io/docs/sol2ink).
Copy link
Contributor

Choose a reason for hiding this comment

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

This repo is not maintained. Do we have a more recent version or an alternative?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don’t know the alternatives

Copy link

Choose a reason for hiding this comment

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

We could remove this pointers for now.

- **Amount:** Up to $250,000
- **Requirements:** 5 approvals (for >$100k: Smart Contract Bounty Council approval + Pitch call by request)
- **Requirements:** 5 approvals (for >$100k: Smart Contract Bounty Council approval + optional pitch call)
Copy link
Contributor

Choose a reason for hiding this comment

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

Who is the Smart Contract Bounty Council?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The curators are

README.md Outdated Show resolved Hide resolved
- Accepted grant applications can be amended at any time. However, this _necessitates a curators reevaluation_ and the same number of approvals as an application. If your application has been accepted and, during development, you find that your project significantly deviates from the original specification, please open a new pull request that modifies the existing application. This also applies in case of significant delays.
- If your _delivery schedule_ significantly changes, please also open a pull request with an updated timeline.
- If your deliveries are significantly delayed and we cannot get a hold of you, we will terminate the grant (4 approvals required. If Curator creates the termination PR, only 3 more approvals are required).
Milestones are to be delivered in the [Grant Milestone Delivery](https://github.com/smart-contract-bounty/Grant-Milestone-Delivery) repository.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Milestones are to be delivered in the [Grant Milestone Delivery](https://github.com/smart-contract-bounty/Grant-Milestone-Delivery) repository.
Milestones are to be delivered in the [Grant Milestone Delivery](https://github.com/use-inkubator/Grant-Milestone-Delivery) repository.


## :bulb: Help
### Real-time conversation
**💡 Help**
Copy link
Contributor

Choose a reason for hiding this comment

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

see above


- [Ecosystem Grants Community](https://matrix.to/#/#sm-bounty-support:matrix.org)

Other channels
Additional channels:

- [Smart contracts & Parity's ink!](https://matrix.to/#/#ink:matrix.parity.io)
Copy link
Contributor

Choose a reason for hiding this comment

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

see above

- [Smart contracts & Parity's ink!](https://matrix.to/#/#ink:matrix.parity.io)
- [Brushfam ink! community](https://matrix.to/#/!utTuYglskDvqRRMQta:matrix.org?via=matrix.org&via=t2bot.io&via=web3.foundation)
- [Brushfam ink! community](https://discord.gg/9FRETSPmp9)
Copy link
Contributor

Choose a reason for hiding this comment

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

This looks abandoned.

Co-authored-by: Sebastian Müller <[email protected]>
Copy link

@al3mart al3mart left a comment

Choose a reason for hiding this comment

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

I feel that bounty levels 1 & 2 can be merged into one.
Maybe above them there should be a level that aligns with the max stated in the proposal and we let teams reach out in case the provided levels don't fit their needs ?

Have left a few nits.


**Technical Showcase**
We recommend using [Sol2ink](https://github.com/Brushfam/sol2ink) to accelerate development. Learn more in the [Sol2ink docs](https://learn.brushfam.io/docs/sol2ink).
Copy link

Choose a reason for hiding this comment

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

We could remove this pointers for now.

Comment on lines 106 to +110
- [OpenBrush](https://openbrush.io/)
- [Swanky](https://github.com/AstarNetwork/swanky-cli)
- Oracles (price feed, VRF, customizable oracles, ...)
- Automation & account abstraction
- DAO toolings (smart contract implementations, UI, SDKs)
- Bridges (XCM integration, and bridges to external blockchains)
- Oracles (price feeds, VRF, customizable oracles, etc.)
- DAO tooling (smart contracts, SDKs, UIs)
- Bridges (XCM integration, external blockchain bridges)
Copy link

Choose a reason for hiding this comment

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

We might want to remove these concrete links from here too.
Some general ideas like DEXs and the listed ones are good enough until we have proper RFCs


- **Hang Yin from Phala Network**: Co-founder and CTO at Phala Network. 5 years of Web3 core development and entrepreneur experience. Author of Phala Network Technical Whitepaper. Lead the team to build an off-chain computing protocol based on ink! and Substrate with 30k online servers. Ex-Googler on Machine Learning before joining Web3.
Community support channels:
Copy link

Choose a reason for hiding this comment

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

- [Michi](https://github.com/cmichi)
- [Velocity Labs](https://www.velocitylabs.org/)
- [David Semakula](https://davidsemakula.com/)
- [R0GUE](https://github.com/al3mart)

Choose a reason for hiding this comment

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

Suggested change
- [R0GUE](https://github.com/al3mart)
- [R0GUE](https://r0gue.io/)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants