-
Notifications
You must be signed in to change notification settings - Fork 529
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
Anagolay project Idiyanale milestone 2 #453
Anagolay project Idiyanale milestone 2 #453
Conversation
Not sure what is happening with the checks, i followed the guidelines. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See comment ⬇️
Co-authored-by: S E R A Y A <takahser@users.noreply.github.com>
@takahser suggestion implemented |
@woss cool, it worked :) We will look into your delivery as soon as possible. Please excuse us if this might take longer than usual, we are currently experiencing a bit of a backlog. |
thanks, I hope you will do it by Monday because we are hoping to have the announcement published on your announcement day 13th of June😊 |
@woss unfortunately, most probably this won't be the case. In any way, if you're planning to publish announcements please make yourself familiar with our announcement guidelines before publishing :) |
We did and we got the announcement approved by your team, only waiting for the green light on this :) |
Hey @cruikshankss I'd like to mention that we fixed a bug in this commit anagolay/w3f-grant-support-repo@5ce0f1c which caused publish service to panic and crash, which was caused by the If you do have any issues please don't hesitate to contact me so i can help 🙌 |
@woss Sorry for the huge delay. To potentially speed it up for you, I am wondering: are you available for a video call demo today or tomorrow? My timezone is MST (USA). If not, no worries. Fyi, it usually takes me much longer to understand and test myself (up to a week, or even more). Again, sorry for the delay. |
Hi @cruikshankss I am Barcelona based which is CET, which means you are 9 hours behind my TZ now. Shall we agree to have a soft plan for a call as soon as you can in the morning? For example, your 9 AM is my 6 PM, if we cannot make it tomorrow then Monday same time? |
Hi @woss! Thanks for being open to a call so soon! My work hours are 1pm-9pm CET. Shall we plan on 1pm CET tomorrow? What is the best way to contact you to set up a call? My email is ashleyg@web3.foundation if you'd like to set up a google calendar event/google meet I do have a meeting 2:30-3pm CET tomorrow, but other than that, feel free to choose any time tomorrow! |
Hi @cruikshankss, those are early hours for you :) email sent with google talk invite |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a quick comment here as well:
- There is a todo here that seems to be missing: https://github.com/anagolay/w3f-grant-support-repo/tree/project-idiyanale-phase1_milestone-2#anagolay-cli-workflow-manifest-generation
- Where can find the testing guide on how to run the unit tests: https://github.com/anagolay/w3f-grant-support-repo/tree/project-idiyanale-phase1_milestone-2
- Could you please link directly to the implementation and not just the repo.
Dear @Noc2 ,
Well spotted, although the context was the Article that was provided. Fixed.
If you refer to the unit tests for the anagolay runtime, you need to clone https://github.com/anagolay/anagolay-chain/tree/w3f-project-idiyanale-phase1_milestone-2 then set it up then run the tests. Nothing different than any other substrate-based chain. The support repo is not meant to provide user with building the runtime, it comes prebuilt.
I do not understand this, what do you refer as |
Hi @woss, he means a link to the benchmarking and a link to the pallet. Could you please provide a link directly to the pallet code within your codebase? Thank you! |
Hi @cruikshankss, i don't understand, you already provided the link and the links to each of the pallets is in the deliverable. The pallets are written in the same way as any other substrate pallet, containing the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @woss thanks for the quick reply. Currently the table of this pr simply links to the support repo for each deliverable and not to the specific deliverables. For example Substrate module: an_workflow should ideally link to the implementation of the module and not just the support repo.
There was a problem hiding this 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. That was what I was looking for. I’m happy to accept and merge this delivery as soon as @cruikshankss merges her evaluation. I just pinged her.
Btw. also someone external took a look at your delivery, you can find his notes here: #482 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I’m happy to confirm that the milestone is accepted. I suggest to wait with the payment for now, because of w3f/Grants-Program#1010
Good morning @woss, Please see the following link for payment confirmation: https://etherscan.io/tx/0x93f383bfd58e98dc03b1567edf615aaeee99b8d50949d8f346fd4c3de83ff4b3 Please let us know if you received the payment. Many thanks, |
Good day @fededubbi , i acknowledge that i have received the payment. Thank you |
Update ZeroDAO_Network.md update Update ZeroDAO_Network.md
Milestone Delivery Checklist
Link to the application pull request: w3f/Grants-Program#719