Skip to content

Polkadot Light Client Milestone #1#579

Merged
Noc2 merged 2 commits intow3f:masterfrom
vbar:master
Dec 2, 2022
Merged

Polkadot Light Client Milestone #1#579
Noc2 merged 2 commits intow3f:masterfrom
vbar:master

Conversation

@vbar
Copy link
Copy Markdown
Contributor

@vbar vbar commented Sep 27, 2022

Milestone Delivery Checklist

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

@FlorianFranzen
Copy link
Copy Markdown

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

Should be w3f/Grants-Program#968

@vbar
Copy link
Copy Markdown
Contributor Author

vbar commented Sep 30, 2022

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

Should be w3f/Grants-Program#968

yes, fixed

@bhargavbh
Copy link
Copy Markdown
Contributor

Apologise for the delay in evaluating. Please find my evaluations here: #603

Would be great if you could address the issues raised, thanks.

@bhargavbh
Copy link
Copy Markdown
Contributor

Please find the evaluation for the revised delivery here: #625

Overall remark:
The concerns raised in previous evaluation have been by and large addressed. However, the delivery is more of an implementation roadmap than a Light Client Spec that can be integrated into Polkadot Spec. We have a few minor suggestions mentioned above, please include them. We have also taken into consideration the initial strides towards implementing a Light Client here. We suggest integrating it into your delivery and making necessary amendments (e.g removing claims of integration into Polkadot Spec) to the initial Grant proposal via a PR.

Copy link
Copy Markdown
Contributor

@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.

Sorry for the delay here. I'm happy to confirm that the milestone is a pass and I will forward your invoice internally, see #645

@Noc2 Noc2 merged commit 34d005c into w3f:master Dec 2, 2022
@RouvenP
Copy link
Copy Markdown

RouvenP commented Dec 13, 2022

hi @vbar we transferred the payment today.

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.

5 participants