Skip to content

Update: Project Aurras - MVP - Phase 2#1402

Merged
takahser merged 5 commits intow3f:masterfrom
MuhammedIrfan:master
Jan 4, 2023
Merged

Update: Project Aurras - MVP - Phase 2#1402
takahser merged 5 commits intow3f:masterfrom
MuhammedIrfan:master

Conversation

@MuhammedIrfan
Copy link
Copy Markdown
Contributor

Update #814

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Dec 28, 2022

CLA assistant check
All committers have signed the CLA.

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.

Thanks a lot for the amendment. Could you also update the original timeline/estimated duration?

@Noc2 Noc2 self-assigned this Dec 29, 2022
@Noc2 Noc2 added the changes requested The team needs to clarify a few things first. label Dec 29, 2022
@MuhammedIrfan
Copy link
Copy Markdown
Contributor Author

Thanks a lot for the amendment. Could you also update the original timeline/estimated duration?

Sure, I have updated. Thank you.

Noc2
Noc2 previously approved these changes Dec 29, 2022
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.

Thanks for the update.

@Noc2 Noc2 added ready for review The project is ready to be reviewed by the committee members. amendment This PR proposes changes to an existing application. and removed changes requested The team needs to clarify a few things first. labels Dec 29, 2022
Copy link
Copy Markdown
Contributor

@alxs alxs left a comment

Choose a reason for hiding this comment

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

Thanks, but could you also update the estimated duration (in time, not just person days) or just add an estimated delivery date to each milestone? It's not clear when you want to deliver the first milestone since it still says 30 working days.

Comment on lines +244 to +249
| Number | Deliverable | Specification | Link |
| ------------- | ------------- | ------------- | ------------- |
| 0a. | staking and payout features for [scs/substrate-api-client](https://github.com/scs/substrate-api-client) | staking functionalities and payout example. fixed bugs in staking modules | https://github.com/scs/substrate-api-client/pull/294 |

- **Estimated Duration:** 45 Working days
- **FTE:** 1 No newline at end of file
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

What is this and how is it related to the rest of the project? If you want to deliver it with the next milestone, could you add it to its deliverable table?

Copy link
Copy Markdown
Contributor Author

@MuhammedIrfan MuhammedIrfan Jan 2, 2023

Choose a reason for hiding this comment

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

We have used scs/substrate-api-client as a wasm-compatible lib for the workflow to interact with substrate-based chains. We have made some contributions that would be needed for our use case. We will add this to the deliverables. Thank you.

@MuhammedIrfan
Copy link
Copy Markdown
Contributor Author

Thanks, but could you also update the estimated duration (in time, not just person days) or just add an estimated delivery date to each milestone? It's not clear when you want to deliver the first milestone since it still says 30 working days.

Sure @alxs , I will update the application with the estimated delivery date

@Noc2
Copy link
Copy Markdown
Contributor

Noc2 commented Jan 2, 2023

@MuhammedIrfan feel free to ping me once it's updated.

@MuhammedIrfan
Copy link
Copy Markdown
Contributor Author

@MuhammedIrfan feel free to ping me once it's updated.

I have updated. Kindly review.

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.

Thanks for the update. I'm happy to go ahead with it and share it again with the rest of the team.

@takahser takahser merged commit 001aa11 into w3f:master Jan 4, 2023
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