-
Notifications
You must be signed in to change notification settings - Fork 29
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 PIV cert generation wizard #88
Conversation
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.
Cool. Some comments:
I think the flow should be restructured a bit, my suggestion is this:
1. Select output format
2. Select algorithm
3. Select subject name
4. IF self-signed: Select expiry date
5. Confirmation view with finish button
6. IF certificate already in slot: Overwrite-confirmation-popup
7. PIN/MGM key popup.
8. IF CSR: Save output file dialog
9. Generic success popup
- Expiration date: should have a default value of 1 year
- Expiration date: don't allow dates before today
- Calendar widget should be aligned to the right of the input field
- Algorithm: ECCP-384 is not supported on NEO
import QtQuick.Controls 2.2 | ||
import QtQuick.Layouts 1.3 | ||
import QtQuick.Controls.Material 2.2 | ||
import Qt.labs.calendar 1.0 |
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 calendar qml module needs to be added as a dependency in debian/control and doc/development
FileDialog { | ||
id: selectCsrOutputDialog | ||
title: "Please choose a destination" | ||
defaultSuffix: "csr" |
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.
Default suffix was added in Qt 5.10 unfortunately.
http://doc.qt.io/qt-5/qml-qtquick-dialogs-filedialog.html#defaultSuffix-prop
Calendar: Drop WeekNumber and DayOfWeek, hard code en_US locale |
piv: small layout fixes for wizard
This is implicit now that the RadioButtons share the same parent.
Otherwise it's too narrow (on my machine) and cuts the content off early...
Polish cert wizard layout
This reverts commit 292f881.
piv: update CSR save dialog
piv: use username as default subject name
calendar: don't allow flickering and flatten buttons
Put NextButton icon on the right
🎉 |
No description provided.