-
Notifications
You must be signed in to change notification settings - Fork 45
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
Extract PasswordField from FromPrivateKey #1018
Conversation
MegaLinter status: ✅ SUCCESS
See errors details in artifact MegaLinter reports on CI Job page |
Codecov Report
@@ Coverage Diff @@
## master #1018 +/- ##
==========================================
+ Coverage 88.78% 88.85% +0.06%
==========================================
Files 100 101 +1
Lines 1748 1750 +2
Branches 405 405
==========================================
+ Hits 1552 1555 +3
+ Misses 196 195 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
|
3b98bf5
to
9a77a7b
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.
Generally, LGTM.
As I have explained, I would switch the icons around, but it's just a small things.
</Tip> | ||
</Box> | ||
</FormField> | ||
></PasswordField> |
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.
This could be a self closing tag.
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.
Changed. I don't really care tho
If it is good practice then we should add react/self-closing-comp
9a77a7b
to
e0855a5
Compare
d6a2a2d
to
bd32f34
Compare
Just making it reusable for persisted state