-
Notifications
You must be signed in to change notification settings - Fork 7
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
Gasless voting UI #2692
Gasless voting UI #2692
Conversation
Deploying decent-interface with
|
Latest commit: |
efd973f
|
Status: | ✅ Deploy successful! |
Preview URL: | https://d22099e8.decent-interface.pages.dev |
Branch Preview URL: | https://eng-25-gassless-voting-ui.decent-interface.pages.dev |
src/components/DaoCreator/formComponents/AzoriusTokenDetails.tsx
Outdated
Show resolved
Hide resolved
0b38204
to
d168e33
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.
I take it screenshots are not accurate anymore in the description. as DAOCreate doesn't have the address anymore?
Gonna mark this as draft since there's a number of unresolved points pending input |
@DarksightKellar The button seems very small and looks weird bellow the divider. @nicolaus-sherrill will be able to give you more input. |
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.
The "Propose" button in Settings is a global button for the settings experience and should be anchored to the bottom of the UI. It is pressed when the user finishes a session and pushes their settings and governance updates to a proposal.
…ag and gas tank address from external source of truth
![]() |
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.
@DarksightKellar this looks great for setup. High-fidelity UI is incoming today.
if (gaslessVotingChanged) { | ||
changeTitles.push(t('enableGaslessVoting', { ns: 'proposalMetadata' })); | ||
|
||
// @todo Is KV pairs the place we're storing this flag? |
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.
Thats a good question? Probably right?
…ToggleDAOCreate` and `GaslessVotingToggleDAOSettings`
Closes ENG-25
Closes ENG-153
DAO settings:
Create DAO flow
Design (implemented in a previous commit):

Implemented currently:

See this comment
See issue for testing instructions