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
[frontend] fix import openpgp manager #1507
Conversation
refactor double declaration of `generateKey`
not fully verified yet |
- refactor in tsx - split forms: generate & import - fix saving pub key
should be fine now |
this.state.files.map((file) => ( | ||
<File file={file} onRemove={this.resetForm} /> | ||
this.state.files.map((file, i) => ( | ||
<File key={i} file={file} onRemove={this.resetForm} /> |
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 array index in key… Mais que fait eslint
? :p
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.
at least index is better than nothing :)
|
||
{keyDetails && ( | ||
<div className="m-openpgp-key__summary"> | ||
<span>{keyDetails?.userIds}</span> |
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.
userIds
are thrown as is, without separators, which is difficult to read. We may want to dive in a little bit to see how to display this in a better way.
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.
fixed
refactor double declaration of
generateKey