-
Notifications
You must be signed in to change notification settings - Fork 298
[DDW-1222] Catalyst UI improvements #3136
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
[DDW-1222] Catalyst UI improvements #3136
Conversation
|
|
||
| ### Features | ||
|
|
||
| - Show "To be defined" if date of the next Catalyst fund is in the past ([PR 3105](https://github.com/input-output-hk/daedalus/pull/3105)) |
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.
Replacing CHANGELOG entry on purpose - effectively, what was delivered in this unreleased PR, was deleted
...p/components/voting/voting-registration-wizard-steps/VotingRegistrationStepsEnterPinCode.tsx
Outdated
Show resolved
Hide resolved
...r/app/components/voting/voting-registration-wizard-steps/VotingRegistrationStepsRegister.tsx
Outdated
Show resolved
Hide resolved
tomislavhoracek
left a comment
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.
@marcin-mazurek nice job 💯
|
Credits to @lucas-barros too ;) |
DominikGuzei
left a comment
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.
much better like this 👏
|
@marcin-mazurek this looks ok overall, but there is a detail with the default name of the pdf file on the last step. |
|
@gabriela-ponce good point, I've replaced "fund" with "catalyst" - can you re-check? |
…atalyst-ui-improvements
|
Hi @marcin-mazurek |
|
@miorsufianiohk I asked Catalyst team to review all links, seems like there is more links that require updating. Good catch! |
miorsufianiohk
left a comment
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.
LGTM. Great work @marcin-mazurek 👍

This PR improves the Catalyst voting UI by removing confusing next fund snapshot date (we will only show info around the current fund) as well as removes any mentions of the fund number from the registration flow (since the user registers for Catalyst voting in general and not for a specific fund).
Screenshots
Testing Checklist
Review Checklist
Basics
input-output-hk/daedalus-devandinput-output-hk/daedalus-qaassigned as PR reviewersrelease-vNext,feature/bug/chore,WIP)yarn manage:translationsproduces no changes)yarn storybook)yarn.lockfile is updatedCode Quality
Testing
After Review