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

pallet-verifier (milestone 1 & 2) #1222

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

Conversation

davidsemakula
Copy link
Contributor

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, invoices must be submitted and payments will be transferred to the Polkadot AssetHub and/or 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#2234 < please fill this in with the PR number of your application.

@PieWol
Copy link
Member

PieWol commented Nov 14, 2024

Hey @davidsemakula ,
thanks for submitting your delivery. I'd like to ask you to either submit the deliveries for individual milestones separately or point out which work has been done to achieve each milestone. In it's current format it's hard to differentiate the work that went towards each milestone. This would make it easier for me to review. Best ✌️

@PieWol PieWol self-assigned this Nov 14, 2024
@davidsemakula
Copy link
Contributor Author

Hi @PieWol

I'd like to ask you to either submit the deliveries for individual milestones separately or point out which work has been done to achieve each milestone.

I'll do the latter and ping you. 🙂

@PieWol
Copy link
Member

PieWol commented Nov 18, 2024

Great, thank you :)

@davidsemakula
Copy link
Contributor Author

Hi @PieWol

point out which work has been done to achieve each milestone.

I've added the details to the delivery document.

@PieWol
Copy link
Member

PieWol commented Nov 22, 2024

Great, thanks. I'll let you know once I made progress on the review.

@davidsemakula
Copy link
Contributor Author

@PieWol I pushed a few updates, please pull the latest from master before reviewing 🙂

@PieWol
Copy link
Member

PieWol commented Nov 26, 2024

Thanks for the update @davidsemakula :)

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