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

Amend the proposal for Spacewalk bridge #1087

Merged
merged 1 commit into from
Aug 4, 2022
Merged

Conversation

TorstenStueber
Copy link
Contributor

Project Abstract

This is an amendment for the proposal for Spacewalk. It has been approved in PR #749.

The project consists of three milestones. We finished the first milestone and it has been approved.

This amendment reflects what we learned while we were working on this project and incorporates feedback from the reviewer of the first milestone. It contains two changes:

  • swapping the order of the 2nd and 3rd milestones (the specification and deliverables of these milestone don't change)
  • extending the timeline and person-hours required for the 2nd and 3rd milestones

The deliverables of the 2nd and 3rd milestones are independent. We have been working in parallel to explore some open questions regarding the 3rd milestone (Stellar oracle). We made good progress and will finish this milestone before we finish our work on the previous 2nd milestone. For that reason we would like to swap both milestones.

It became apparent that we underestimated the amount of work required for both the 2nd and 3rd milestones. First, the previous 2nd milestone is heavily based on the fairly complex interBTC bridge. It requires a lot of effort to navigate, understand and adapt that codebase to our purposes. The previous 3rd milestone required some extensive explorative pioneering work that we conducted with support of the Stellar core developer team.

For which grant level are you applying?

  • 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)

Application Checklist

  • The application template has been copied, renamed ( project_name.md) and updated.
  • I have read and understood the FAQs, application guidelines and announcement guidelines.
  • A BTC, Ethereum (USDT/USDC/DAI) or Polkadot/Kusama (aUSD) address for the payment of the milestones is provided inside the application.
  • I have read and acknowledge 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.

@CLAassistant
Copy link

CLAassistant commented Jul 26, 2022

CLA assistant check
All committers have signed the CLA.

@Noc2 Noc2 requested a review from ashlink11 August 3, 2022 07:09
Copy link
Contributor

@ashlink11 ashlink11 left a comment

Choose a reason for hiding this comment

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

Thank you for your amendment. I was impressed with your dedication on Milestone 1 to faithfully and properly implement/rejuvenate the immensely complex interBTC protocol. I think that alone is valuable work for the ecosystem as a whole. I am happy to approve this amendment (one approval) for the additional complexity and costs, and I trust your rearrangement of the milestone order. I personally think your project has an immense amount of broader ecosystem potential as a parachain, a fiat onramp, and oracle. Also, thank you in general for your computer science work (in the true empirical sense of the phrase) with wasm/Layer 0. I will share this amendment with my colleagues.

@rrtti rrtti self-requested a review August 3, 2022 09:24
@Noc2 Noc2 merged commit bda6629 into w3f:master Aug 4, 2022
@TorstenStueber
Copy link
Contributor Author

Thanks a lot everyone for the positive review and for approving this amendment.

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.

None yet

5 participants