-
Notifications
You must be signed in to change notification settings - Fork 47
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
WIP: Feat: Offchain DAO creation #957
Conversation
70d2a5a
to
ce547ff
Compare
b5c67e2
to
98ec3b1
Compare
1c62dd5
to
f889286
Compare
98ec3b1
to
92665b4
Compare
9bc270a
to
f889286
Compare
0d1bd50
to
cf37dcf
Compare
e5eb97a
to
4527f50
Compare
e9774ec
to
5d23c8f
Compare
38bbc6a
to
65d07a3
Compare
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.
Some of the useMemo
and useCallbacks
used are not necessary. Ensure you try any changes to these methods, since I can only speak for the code I've seen... I tried to check all the affected methods to ensure I'm not saying it wrong, but I could be wrong in some of them... so double-check it please.
const defineCommitteeIsValid = useMemo(() => { | ||
if ( | ||
!committee || | ||
!committee.length || | ||
errors.committee || | ||
errors.committeeMinimumApproval || | ||
errors.executionExpirationMinutes || | ||
errors.executionExpirationHours || | ||
errors.executionExpirationDays | ||
) | ||
return false; | ||
return true; | ||
}, [ | ||
committee, | ||
errors.committee, | ||
errors.committeeMinimumApproval, | ||
errors.executionExpirationMinutes, | ||
errors.executionExpirationHours, | ||
errors.executionExpirationDays, | ||
]); | ||
|
||
const proposalCreationIsValid = useMemo(() => { | ||
// required fields not dirty | ||
// if multisig | ||
if (membership === 'multisig') { | ||
if (!['multisig', 'anyone'].includes(eligibilityType)) { | ||
return false; | ||
} | ||
return true; | ||
} else { | ||
return !errors.eligibilityTokenAmount; | ||
} | ||
}, [eligibilityType, errors.eligibilityTokenAmount, membership]); |
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.
No need to use useMemo
here.
9dfb36b
to
d0e2461
Compare
3723975
to
af00d3e
Compare
d0e2461
to
32d3c90
Compare
32d3c90
to
264140e
Compare
Closed in favour to #1074 |
Description
This PR is developed on the vocdoni
offChain
plugin context.The branch implement changes on the DAO creation form to add the option to create
offChain
proposals with a executive committee.Type of change
Checklist:
CHANGELOG.md
file in the root folder.