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

Polkadart amend #2004

Merged
merged 3 commits into from
Sep 27, 2023
Merged

Polkadart amend #2004

merged 3 commits into from
Sep 27, 2023

Conversation

leonardocustodio
Copy link
Contributor

Lohann has done some amazing work into this lib but unfortunately he has not been able to dedicate the necessary time to continue with the works.

As discussed in the Grant PR proposal #1053 (comment) to continue the development and finish this grant we opted to change the project owner as well as the milestone values and objectives.

We acknowledge we underestimated the effort required to finish the last two milestone. There were 5 engineers working on this but at this stage things got too complex and we remained with only 2 engineers with enough knowledge to finish the work.

Polkadart has been an amazing project and we have already have people using it in production environment. As Flutter together with React Native is one of the most popular mobile frameworks we are very sure that this project is a great contribution to our ecosystem allowing people to develop apps that support substrate based chains in any language they want.

@github-actions
Copy link
Contributor

github-actions bot commented Sep 26, 2023

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@leonardocustodio
Copy link
Contributor Author

I have read and hereby sign the Contributor License Agreement.

Copy link
Collaborator

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

Thanks for the PR. Could you also update the total costs?

@Noc2 Noc2 added the changes requested The team needs to clarify a few things first. label Sep 26, 2023
@semuelle semuelle self-assigned this Sep 26, 2023
@semuelle
Copy link
Member

For reference, Lohann has given his approval here.

Copy link
Member

@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 for the amendment, @leonardocustodio. I'm generally happy to approve it. Could you clarify the timeline, though? You are, if I'm reading this right, planning to submit M3 in six months and requesting 40k for 2 full-time developers for the next ten months?

applications/Polkadart.md Outdated Show resolved Hide resolved
Co-authored-by: Sebastian Müller <[email protected]>
@leonardocustodio
Copy link
Contributor Author

Hello @semuelle,

Sure, milestone 3 is already halfway through I've increased it to 6 months to consider the time that was already spent and to have the project proposal updated with the whole time spent and the whole investment made. The delivery of milestone 3 should happen in 2 to 3 months. So I would say we "only" have about 6 months ahead to finish the whole project.

@keeganquigley keeganquigley added the amendment This PR proposes changes to an existing application. label Sep 27, 2023
Copy link
Member

@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. I think the added scope seems quite useful, so happy to approve.

@semuelle semuelle added ready for review The project is ready to be reviewed by the committee members. and removed changes requested The team needs to clarify a few things first. labels Sep 27, 2023
Copy link
Contributor

@keeganquigley keeganquigley left a comment

Choose a reason for hiding this comment

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

Thanks @leonardocustodio we appreciate it, LGTM.

@takahser takahser merged commit c95301a into w3f:master Sep 27, 2023
11 checks passed
@leonardocustodio leonardocustodio deleted the patch-1 branch February 23, 2024 22:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
amendment This PR proposes changes to an existing application. ready for review The project is ready to be reviewed by the committee members.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants