near-daos / sputnik-dao-contract

Smart contracts for https://app.astrodao.com
https://astrodao.com/
MIT License
107 stars 79 forks source link

Updating policy partially via JSON paths #62

Closed ilblackdragon closed 2 years ago

ilblackdragon commented 2 years ago

Example to just rename 0th role to test: update_policy '{"updates": [{"path": "roles/0/name", "new_value": "\"test\"\"}]}'

ctindogaru commented 2 years ago

Hey Illia, to solve some of the issues mentioned in this PR, I've tried to come up a simpler approach in https://github.com/near-daos/sputnik-dao-contract/pull/88. Please let me know what you think.

ctindogaru commented 2 years ago

Merged https://github.com/near-daos/sputnik-dao-contract/pull/88 instead. Although your implementation is brilliant, I find that #88 is easier to use for the end user and it also allows you to add/remove new groups from the policy.