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

QSTN - Milestone 1 (Delivery).md #1017

Closed
wants to merge 6 commits into from
Closed

QSTN - Milestone 1 (Delivery).md #1017

wants to merge 6 commits into from

Conversation

qstnus
Copy link
Contributor

@qstnus qstnus commented Sep 28, 2023

Milestone Delivery Checklist

  • The milestone-delivery-template.md has been copied and updated.
  • This pull request is being made by the same account as the accepted application.
  • I have disclosed any and all sources of reused code in the submitted repositories and have done my due diligence to meet its license requirements.
  • In case of acceptance, an invoice must be submitted and the payment will be transferred to the BTC/ETH/fiat account provided in the application.
  • The delivery is according to the Guidelines for Milestone Deliverables.

Link to the application pull request: w3f/Grants-Program#1082

@qstnus qstnus changed the title QSTN - Milestone 1 (Delivery) QSTN - Milestone 1 (Delivery).md Sep 28, 2023
@keeganquigley
Copy link
Contributor

Thanks @qstnus moving the conversation here, and thanks for the delivery. I updated the text to link to your original proposal's pull request. Can you also make the following changes?

  • The Application Document: towards the top should link to your application, which in this case the link is https://github.com/w3f/Grants-Program/blob/master/applications/QSTN.md
  • "Milestone 1 Example" should be changed to "Milestone 1" (above the delivery tables).
  • Usually the links should be included in the tables themselves, but since you have all the links in the text above I think this should be okay. Optionally you could add links to each section of the text.

Thanks!

@qstnus
Copy link
Contributor Author

qstnus commented Sep 28, 2023

Hello @keeganquigley,

Thanks for this information. The link has been added to the top. In terms of the deliverables, I kept the same original presentation but the order of information and flow is the same as the original grant milestone.

Looking forward to next steps!

@keeganquigley
Copy link
Contributor

Thanks @qstnus I updated it to get rid of the template text. Looks better now.

@keeganquigley
Copy link
Contributor

Thanks again @qstnus I have started the evaluation, and I have the following comments:

  • It's great to see that the app is live on Moonbeam, but the deliverables for this grant include custom pallets, presumably built into a local substrate node. Can you link me to these? Or am I to understand that these aren't built, as you pivoted to building Solidity contracts on Moonbeam instead?
  • These changes aren't mentioned in the application, and I'm not seeing that any amendments were filed, which would be required to document these changes since they are rather significant. Were these talked about at some point?

@qstnus
Copy link
Contributor Author

qstnus commented Oct 4, 2023

Hello @keeganquigley ,

Thank you for the feedback. Post our engagement with engineers from the Polkadot Blockchain Academy, we opted to pivot to accelerate our time to market by transitioning to Solidity contracts on Moonbeam. This shift aligns better with our new multi-chain approach across EVM-compatible chains, facilitating business funding within our survey module.

I acknowledge the oversight in not updating our initial application. I seek guidance on amending our proposal and articulating the deliverables to align with Polkadot’s expectations. Your insight would be invaluable.

@keeganquigley
Copy link
Contributor

Thanks for confirming @qstnus. After discussing the matter internally with the committee, I must inform you that, unfortunately, we can't accept the delivery as is, since it doesn't meet the technical requirements originally agreed upon.

The reason we ask for amendments is to discuss significant changes such as these, because I would have been able to tell you earlier on that generally, our grants program doesn't support Solidity or Moonbeam development. This is partly because we aren't experts in the EVM field, and also because Moonbeam has its own grants program which caters to these types of projects.

Therefore, I believe this situation leaves two options:

  • If you are still interested in developing the substrate pallets/modules, you could file an amendment to extend the timeline and continue to work on the grant as is. Although I will point out that this would need two approvals from the committee, who will likely take this situation into account.
  • Alternatively, we could go ahead and reject the delivery and close the grant. That way you could try other funding means such as the Moonbeam grants program, although I'm not sure if they offer retroactive payments or not.

Please let me know and thanks for your understanding.

@qstnus
Copy link
Contributor Author

qstnus commented Oct 4, 2023

Hello @keeganquigley ,

Thanks for this information. I spoke with our CTO and we can deliver the original scope in a few weeks. We will keep our Moonbeam implementation and proceed with the custom pallet built into a local substrate node.

The original application has been amended. In regards to our commitment to DOT, we are excited to continue building in the ecosystem and applied for the Polkadot Blockchain Academy. Please let me know the next steps!

@keeganquigley
Copy link
Contributor

Thanks @qstnus however I'm not seeing an amendment. Did you submit a PR here?

@qstnus
Copy link
Contributor Author

qstnus commented Oct 4, 2023

Hello @keeganquigley,

Sorry about that. I made the PR and signed the CLA agreement. Let me know if you have any questions. Thanks!

@keeganquigley
Copy link
Contributor

Hi @qstnus just checking if you are you still on track to submit this by the end of this week?

@qstnus
Copy link
Contributor Author

qstnus commented Oct 23, 2023

Hello @keeganquigley,

To my understanding - we extended the timeline to December based on the last revision with the W3F. Please confirm!

P.S. The timeline is 15 months - we received initial approval October 2022

Link: w3f/Grants-Program#2030 (comment)

@keeganquigley
Copy link
Contributor

Thanks @qstnus sounds good. I see that three months were added in the amendment. In that case I will go ahead and close this pull request and you can re-open a new one for the amended delivery. Thanks!

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.

2 participants