-
Notifications
You must be signed in to change notification settings - Fork 30
Feature: EthHasInfo add showHash property #120
Conversation
ESLint Summary View Full Report
Report generated by eslint-plus-action |
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.
Nice!
Please update the snapshots.
@nicosampler , could you provide more details about where and how it influences on the app ? It really hard to understand how and where to test. Why do we change it ? Is it related to some issue? |
Travis automatic deployment: |
I just noticed some errors in stories about styled-components usage: how did this get through? |
@liliya-soroka, this is a stand-alone component. Not directly related to safe-react repository. |
1cbb25d
to
fcedd67
Compare
I've filed #122 to fix the tests. |
Travis automatic deployment: |
1 similar comment
Travis automatic deployment: |
showHash
prop.border-radius
of 50% was breaking some SVGs. If that is needed should be customized when the Component is being used.