-
Notifications
You must be signed in to change notification settings - Fork 59
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
Add templates to installer descriptor editor #1366
Add templates to installer descriptor editor #1366
Conversation
29a1320
to
41ec55c
Compare
a4bf658
to
b49de79
Compare
b2f70bc
to
9f0e32f
Compare
273814c
to
05d6e1e
Compare
If I choose the "Custom wallet" option and then click the trash icon next to "Primary key" twice, I get the following error:
|
0ef2df4
to
0cd79b7
Compare
Tested on 0cd79b7: Unlike in v7, the fingerprint is not cleared if I change the signing device: |
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 comments about the wording.
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 think it should be "Receipt of funds" (in both images).
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 updated the figmas with "Receipt".
PS: The updated svgs will be given once the templates and wording discussion will be finalized.
0cd79b7
to
c90d9db
Compare
c90d9db
to
294e4ba
Compare
71358a6
to
bd03cc9
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.
Self-ACK bd03cc9
Tested multiple time by me and Michael, next iteration should be in an other PR.
WIP for #1147