thetta / dao-templates Goto Github PK

Основная идея DAICO шаблона в том, что он совершает пошаговые выплаты на основании прошедшего ревью. Есть эксперты, которые и проводят такое ревью. Если проект не выполнил предыдущий этап, то выплаты прекращаются.

Логика работы

TODO (пока не готово)

вообщем так и не поправил этот баг
when "add new group member" voting is created then on the last vote, that would add team member, metamask consumes huge amount of gas so there is smth wrong

после создания голосования на добавление нового участника через daoBaseAuto.addGroupMemberAuto(), если к примеру есть 3 участника в группе и первый голосует "за", то при голосовании второго выскакивает revert

truffle debug ссылается на:

263:   * @dev this function issue tokens for address _to
264:   */
265:   function issueTokens(address _tokenAddress, address _to, uint _amount) public isCanDo(ISSUE_TOKENS) {
       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

debug(development:0x79ba65ba...)> 

Transaction halted with a RUNTIME ERROR.

This is likely due to an intentional halting expression, like assert(), require() or revert(). It can also be due to out-of-gas exceptions. Please inspect your transaction parameters and contract code to determine the meaning of this error.```
насколько я понимаю isCanDo возвращает false и поэтому выскакивает revert, но причем здесь permission на ISSUE_TOKENS я так и не понял, в теории чтобы добавить нового участника в группу этот permission вообще не нужен

вполне возможно, что в core то же самое, так как нет функциональных тестов, которые проверяют "полный круг" голосования: создание => голосование => проверка результатов, такие тесты по идее должны быть тут https://github.com/Thetta/Thetta-DAO-Framework/blob/master/test/daobase_auto.functional.tests.js

был такой таск https://github.com/Thetta/Thetta-DAO-Framework/issues/277, но он теперь закрыт, вероятно следует его открыть заново

из полезного поправил небольшие косяки на фронте DevZenDao и добавил читабельные параметры для каждого предложения

в тулбаре данные не сразу подгрузились
пока не сделал F5

"параметры"

в http://localhost:4200/dashboard/dao-param-page
нужно подгружать сюда текущие данные

“команда”

http://localhost:4200/dashboard/team-page

1. поля желательно сделать на всю ширину, как в окне "параметры". Сейчас адрес не войдет по ширине -

2. "Следующий организатор" -> "Установить следующего ведущего"
3. "Гость пришел" -> "Начислить репутацию гостю"
4. "Вывод всех ETH" -> "Вывести собранные ETH на адрес:"
5. "Запустить следующий эпизод" -> "Текущий эпизод завершился"
6. Не понятно, чем "Сменить гостя" отличается от "Экстренно сменить гостя"
   Было бы круто сделать панель с подсказками (текстом)
7. "Сжечь DZT гостя" - насколько я помню, _burnGuestStake() это internal и вызываться напрямую не должна

"патрон"

1. Лучше разделить на 2 отдельных вкладки: "Для слушателя" и "Для рекламодателя"

Для слушателя:
"Стать следующим гостем"
"Купить DZT"

Для рекламодателя:
"Купить рекламное место"

2. Нужно убрать "Прошла ли 1 неделя" вообще
3. Нужно убрать "Аппрув DZT", а делать это автоматически. То есть спрашивать юзера "Чтобы стать гостем, вам нужно положить в стейк N токенов. Вы уверены?"
   и далее автоматом делать 2 транзы: approve + вызов метода

All params (like contract address) should be set in ENV.VARIABLES and then read from config (if no ENV.VARIABLE is set)

1. Use example code here - https://github.com/Thetta/Thetta-DAO-Framework/tree/dev2/contracts/dao_examples

https://github.com/Thetta/Thetta-DAO-Framework/tree/dev2/test/dao_examples

2. Write tests
3. (optional) Write migrations to deploy the selected DAO automatically

i don’t think this variable is needed - https://github.com/Thetta/DAO-Templates/blob/master/contracts/3-DevZenDao/DevZenDaoCore.sol#L40

Because:
https://github.com/Thetta/Thetta-DAO-Framework/blob/master/contracts/DaoClient.sol

See these changes:

Thetta/Thetta-DAO-Framework#102
Thetta/Thetta-DAO-Framework#147 (should be used in DevZenDao template)

чтобы уменьшить размер байт кода, нужно уменьшить кол-во кода из import'ов

конкретно для HierarchyDao можно попробовать:

• создать токены в отдельной миграции
• импортировать абстрактные классы, без реализации
• в конструктор добавить адреса токенов

вообщем можно создать таск вроде "prepare contracts for deployment in test networks" и повесить мне

Holacracy template для ребят
https://hackmd.io/s/BkM44rvcm

Они используют холакратию, но не в “полном” виде, а в весьма урезанном. Поэтому не нужно делать шаблон со всеми фичами холакратии (их очень много и это целый бизнес процесс), а нужно сделать шаблон только для их конкретного use case’а.

TODO

1. Сделать текстовое описание (hackmd файл) логики работы такого DAO шаблона. После этого мне показать текстовый файл, чтобы я мог подтвердить, что все ОК.
2. Сделать сам шаблон (кодом) в DAO-templates

"To become a guest, a listener has to become a patron first (i.e., they have to buy some DevZen tokens), then they must stake S tokens for D days. After the show has ended, S tokens are returned to the patron. If the guest missed the show (that is bad), the tokens are burned. "

TODO

1. Config file format should support different networks:

export const environment = {
	production: false,
  	devZenDaoFactoryAddress: {
             "kovan": "0x42276e2ef62c41580b0c00c5b9566d60ef3a9309",
             "ropsten": "0x4546546545646545646545640121312311231a",
             "rinkeby": "0x24289081092831243243242342342342342341",
             "mainnet": "0x12312398128739012830918239081290381920"
        }

2. Dapp should automatically detect the current MetaMask network and use param from config accordingly

From Slack:

вообщем так и не поправил этот баг
when "add new group member" voting is created then on the last vote, that would add team member, metamask consumes huge amount of gas so there is smth wrong

после создания голосования на добавление нового участника через daoBaseAuto.addGroupMemberAuto(), если к примеру есть 3 участника в группе и первый голосует "за", то при голосовании второго выскакивает revert

truffle debug ссылается на:

263:   * @dev this function issue tokens for address _to
264:   */
265:   function issueTokens(address _tokenAddress, address _to, uint _amount) public isCanDo(ISSUE_TOKENS) {
       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

debug(development:0x79ba65ba...)> 

Transaction halted with a RUNTIME ERROR.

This is likely due to an intentional halting expression, like assert(), require() or revert(). It can also be due to out-of-gas exceptions. Please inspect your transaction parameters and contract code to determine the meaning of this error.```
насколько я понимаю isCanDo возвращает false и поэтому выскакивает revert, но причем здесь permission на ISSUE_TOKENS я так и не понял, в теории чтобы добавить нового участника в группу этот permission вообще не нужен

вполне возможно, что в core то же самое, так как нет функциональных тестов, которые проверяют "полный круг" голосования: создание => голосование => проверка результатов, такие тесты по идее должны быть тут https://github.com/Thetta/Thetta-DAO-Framework/blob/master/test/daobase_auto.functional.tests.js

See this as example: https://github.com/Thetta/Thetta-DAO-Framework/tree/dev2

1. README
2. .gitignore
3. CODE_OF_CONDUCT.md
4. CONTRIBUTING.md
5. DEVELOPMENT.md
6. ISSUE_TEMPLATE.md
7. LICENSE
   ...

See moveToNextEpisode() method.

...

// TODO: implement daoBase.getMemberByIndex() method
address member = 0x0;
...

daoBase.getMemberByIndex is already implemented. You can use it.

Frontend source code is here - https://github.com/Thetta/DAO-FrontendBase

===================================

1. Change theme and components library

• I will take the same theme as we used for CFD project
• I will remove NgMaterial and will use PrimeNg componets(the same library was used for CFD project)
  The reason is that it will take more time to finish and maintain the project with NgMaterial than it would be with PrimeNg

2. Pages structure

• main
  • next episode details
  • dao parameters details
• DAO params where params can be changed(with voting)
• team
  • withdraw ether(creates voting)
  • select the next host(creates voting)
  • change the guest(creates voting)
  • emergency change the guest(creates voting)
  • move to next episode(creates voting)
• advertiser
  • run ads
• listener
  • become the next show guest
  • buy tokens
• group
  • add group member(creates voting)
  • remove group member(creates voting)
• proposals
  • vote

Repo to use - https://github.com/Thetta/DAO-FrontendBase

Лучше, если сделаешь в https://github.com/Thetta/DAO-FrontendBase
какой-то базовый код сначала, от которого потом можно наследоваться другим репозиториям
то есть потом заведем какой-то уже более конкретный https://github.com/Thetta/DevZenDAO от него

По архитектуре я бы делал так:
1) Сначала давай разберемся с фронтом, то есть там должна быть верстка, сборка, логика фронта https://github.com/Thetta/DAO-FrontendBase
2) (УЖЕ ПОСЛЕ ПОСЛЕ)) туда же добавим truffle файлы, чтобы фронт + контракты уже лежали в https://github.com/Thetta/DevZenDAO

то есть чтобы можно было взять потом https://github.com/Thetta/DAO-FrontendBase
и там сразу собирать контракты + тесты + миграции
и собрать фронт опять же оттуда же

сильно не парься с “базовым кодом“, возможно он будет сто раз меняться еще
но в идеале именно такая ситуация, чтобы потом была у нас

when "add new group member" voting is created then on the last vote, that would add team member, metamask consumes huge amount of gas so there is smth wrong

после создания голосования на добавление нового участника через daoBaseAuto.addGroupMemberAuto(), если к примеру есть 3 участника в группе и первый голосует "за", то при голосовании второго выскакивает revert

truffle debug ссылается на:

263:   * @dev this function issue tokens for address _to
264:   */
265:   function issueTokens(address _tokenAddress, address _to, uint _amount) public isCanDo(ISSUE_TOKENS) {
       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

debug(development:0x79ba65ba...)> 

Transaction halted with a RUNTIME ERROR.

This is likely due to an intentional halting expression, like assert(), require() or revert(). It can also be due to out-of-gas exceptions. Please inspect your transaction parameters and contract code to determine the meaning of this error.```
насколько я понимаю isCanDo возвращает false и поэтому выскакивает revert, но причем здесь permission на ISSUE_TOKENS я так и не понял, в теории чтобы добавить нового участника в группу этот permission вообще не нужен

вполне возможно, что в core то же самое, так как нет функциональных тестов, которые проверяют "полный круг" голосования: создание => голосование => проверка результатов, такие тесты по идее должны быть тут https://github.com/Thetta/Thetta-DAO-Framework/blob/master/test/daobase_auto.functional.tests.js

Should be:

1. Если Metamask установлен, но не загружен:
   "Пожалуйста, войдите в MetaMask"
2. Если Metamask вообще не установлен:
   "Пожалуйста, установите MetaMask (ссылка)"

и еще лучше это не диалогом показывать, а сделать отдельные страницы вроде:
/metamask_not_installed
/metamask_not_logged

1. Add createNewContract() to DevZenDaoFactory which creates instance of DevZenDao
2. Inherit DevZenDaoFactoryTestable from DevZenDaoFactory and rewrite createNewContract() which should create instance of DevZenDaoTestable

Notice: if DevZenDaoFactoryTestable is inherited from DevZenDaoFactory the error "Error: The contract code couldn't be stored, please check your gas amount" appears on testing

contract DevZenDaoFactory {
    DevZenDaoCore dao;

     function createDao(){
         createNewContract(); 
         ...
     }

     function createNewContract() internal{
         dao = new DevZenDao();
     }
}

contract DevZenDaoFactoryTestable is DevZenDaoFactory {
    // переопредел только этот метод (то есть 1 строчка!)
    // все остальные методы оставлены такими же
     function createNewContract()  internal {
         dao = new DevZenDaoTestable();
     }
}

TODO

1. Write truffle migration to deploy all libs + DevZenDao contract to testnet
2. Run em

TODO

1. Use example code here - https://github.com/Thetta/Thetta-DAO-Framework/tree/dev2/contracts/dao_examples

https://github.com/Thetta/Thetta-DAO-Framework/tree/dev2/test/dao_examples

2. Write tests
3. (optional) Write migrations to deploy the selected DAO automatically

These folders:

https://github.com/Thetta/Thetta-DAO-Framework/tree/dev2/contracts/dao_examples

https://github.com/Thetta/Thetta-DAO-Framework/tree/dev2/test/dao_examples

1. DAO-Templates should be the 'truffle' repo

# Use this command to init repo ))))
truffle init

2. Move files/tests to https://github.com/Thetta/DAO-Templates to contract and test folders respectively
3. DAO-Templates should use Thetta DAO Frameworks (add 'thetta-dao-core' module to package.json) and put it into node_modules files

# Use this command to install Thetta DAO core
npm install --save thetta-dao-core

4. All links to Thetta DAO Framework contracts should look like this -
   import "thetta-dao-core/contracts/DaoBase.sol";

https://github.com/Thetta/DAO-Templates/blob/master/contracts/3-DevZenDao/DevZenDao.sol

Each method with isCanDo() should have the auto caller:

function updateDaoParams(Params _params) isCanDo("DevZen_updateDaoParams") public {
  super._updateDaoParams(_params);
}

// TODO
contract DevZenDaoAuto is DevZenDao {
   // TODO
   function updateDaoParamsAuto(Params _params) public {}
}
}