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

Create RainbowDAO-Protocol-ink!-Phase1-milestone-1.md #334

Merged
merged 12 commits into from
Jan 17, 2022

Conversation

RainbowcityKun
Copy link
Contributor

@RainbowcityKun RainbowcityKun commented Dec 31, 2021

Milestone Delivery Checklist

Link to the application pull request: w3f/Grants-Program#680 < please fill this in with the PR number of your application.

@RainbowcityKun
Copy link
Contributor Author

Hi, friend of the web3 foundation. In the original application I submitted, there were spaces in the grant name, which caused the current inspection to fail. How do I handle this situation now?

https://github.com/RainbowcityFoundation/Grants-Program/blob/7393e564d87ad2b87e87bae1b584f15d0c5fdc6b/applications/RainbowDAO%20Protocol%20ink!%20%20Phase%201.md

@Noc2
Copy link
Collaborator

Noc2 commented Jan 3, 2022

Thanks for the delivery. We will look into it as soon as possible.The github checks are mostly for us, so don’t worry about it.

@RainbowcityKun
Copy link
Contributor Author

Thanks for the delivery. We will look into it as soon as possible.The github checks are mostly for us, so don’t worry about it.

OK, thank you very much for your reminder. Feel free to contact me if you have any questions.

@takahser takahser self-assigned this Jan 4, 2022
@takahser
Copy link
Contributor

takahser commented Jan 7, 2022

@RainbowcityKun thanks again for the delivery.
After taking a look at it, these are some things I noticed:

  • The GovnanceDAO tests currently fail
  • The docker setup instructions didn't work on my macbook. Also running docker --mirror Aliyun fails with: unknown flag: --mirror
  • I was able to run the frontend code but when testing the protocol functions, all transactions I tried to send failed when using the default settings. Also the instructions are missing on how I can use the UI using a local node.

Could you take a look at these or comment on it?

@RainbowcityKun
Copy link
Contributor Author

Hi @takahser, thank you so much for your hard work, we've updated a few of your questions, please recheck to see if it works.

  • The GovnanceDAO contract file has been updated and you can download it again.

https://github.com/RainbowcityFoundation/RainbowDAO-Protocol-Ink-milestone_1/tree/main/govnance_dao

  • Docker re-updated the documentation, you can see if it works.

https://github.com/RainbowcityFoundation/RainbowDAO-Protocol-Ink-UI-milestone_1/blob/main/docs/docker.md

  • The front-end documentation has also been updated, and the links are as follows:

https://github.com/RainbowcityFoundation/RainbowDAO-Protocol-Ink-UI-milestone_1#readme

Feel free to leave us a comment if you have any questions, thank you again for your hard work.

@takahser
Copy link
Contributor

takahser commented Jan 10, 2022

Hi @RainbowcityKun

Thx again for the updates!

The following works fine now:

  • GovnanceDAO tests

I still have some comments for:

  • Docker docs: instructions on how to start a container from the built image are still missing
  • Rainbow Protocol Tutorial: contrary to the docs, the build.sh script doesn't exists

I'll take a look at the updated frontend docs again after the you add the build.sh script.

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.

And just to add one thing, which I just noticed. The smart contract currently don’t seem to have any inline documentation. See the contract: “We will provide both inline documentation of the code”

@RainbowcityKun
Copy link
Contributor Author

Hi, @takahser ,thank you for your hard work. Now that we've updated these two files again, take a look again.

https://github.com/RainbowcityFoundation/RainbowDAO-Protocol-Ink-UI-milestone_1/blob/main/docs/docker.md

https://github.com/RainbowcityFoundation/RainbowDAO-Protocol-Ink-milestone_1/blob/main/build.sh

If you find any new problems, feel free to contact me and we will improve them as soon as possible. Thanks again for your hard work!

@RainbowcityKun
Copy link
Contributor Author

And just to add one thing, which I just noticed. The smart contract currently don’t seem to have any inline documentation. See the contract: “We will provide both inline documentation of the code”

Hi,@Noc2,thank you very much for your careful inspection. We have now updated all contracts and added inline documentation. Please check again and confirm. If you have any questions, leave me a message at any time, and we will update it as soon as possible.

@takahser
Copy link
Contributor

Thanks again for the update @RainbowcityKun,
I'll take a look at it and give you feedback soon.

Copy link
Contributor

@takahser takahser left a comment

Choose a reason for hiding this comment

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

Hey @RainbowcityKun
Thanks for the updates, I'm now fine with the following:

  • inline comments
  • docker docs
  • build script was added

However, I realised that although you added the instructions for installing a substrate contracts node the following instructions are missing:

  • starting the node
  • connect the local node in the https://polkadot.js.org/apps/
  • the detailed steps on how to get to the interface for uploading the contract (developer > contracts > Upload & deploy code I guess)

I'll go ahead and deploy and test the protocols now before I'll be able to give additional feedback. But feel free to improve the already mentioned points in the meanwhile. :)

@RainbowcityKun
Copy link
Contributor Author

@RainbowcityKun 感谢您的更新,我现在对以下内容很好:

  • 内联评论
  • 码头工人文档
  • 添加了构建脚本

但是,我意识到尽管您添加了安装基板合同节点的说明,但缺少以下说明:

在我能够提供额外反馈之前,我将继续部署和测试协议。但与此同时,请随时改进已经提到的要点。:)

Ok, thank you very much for your careful inspection, and let's keep improving our documentation. We will continue to update, if you have any new questions, feel free to leave us a message.

@RainbowcityKun
Copy link
Contributor Author

@takahser The tutorial files are further updated, please check again, thank you again for your hard work.

https://github.com/RainbowcityFoundation/RainbowDAO-Protocol-Ink-docs/blob/main/rainbowDaoProtocolTest.md#1-installing-the-substrate-contracts-node

@RainbowcityKun
Copy link
Contributor Author

@takahser , @mmagician ,
Thanks to @takahser for the careful evaluation of our first milestone submission, it also made us realize that there are a lot of small issues, we will double check based on the content of the evaluation report. When committing to milestone 2, we'll tackle all the minor issues together.

@Noc2
Copy link
Collaborator

Noc2 commented Jan 17, 2022

Thanks! I’m happy to confirm that the milestone is approved and I will forward your invoice internally.

@Noc2 Noc2 merged commit 674e50a into w3f:master Jan 17, 2022
@RainbowcityKun
Copy link
Contributor Author

Thanks! I’m happy to confirm that the milestone is approved and I will forward your invoice internally.

Thank you very much for your hard work, we will re-optimize our code. The 2nd milestone will also be submitted as soon as possible. We will strive to build the best DAO infrastructure product.

@RouvenP
Copy link

RouvenP commented Jan 24, 2022

hi @RainbowcityKun we transferred the payment today.

failfmi pushed a commit to LimeChain/Grant-Milestone-Delivery that referenced this pull request Sep 26, 2022
* Open Node Framework

* Update potential ecosystem project integration

* Add Polkadot K8s Monitor

* Refreshed HA choices
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