Skip to content

🐛 Bug Fix for invalid 'showBorder' prop- Closes #32#64

Closed
escottalexander wants to merge 1 commit intoshapeshift:developfrom
escottalexander:WEB-32
Closed

🐛 Bug Fix for invalid 'showBorder' prop- Closes #32#64
escottalexander wants to merge 1 commit intoshapeshift:developfrom
escottalexander:WEB-32

Conversation

@escottalexander
Copy link

Link to issue #32

Description

I removed the showBorder property from the Image element as it is invalid syntax. I verified that nowhere in the application relies on this property in its invalid form or what would be valid as a custom property showborder. I also checked documentation for the Image element and confirmed the property does not exist.

Testing

  1. Got latest from develop branch and ran project locally
  2. Saw the error when the WalletButton component was initialized
  3. Removed invalid property
  4. Re-ran application and confirmed no side effects from change

@escottalexander escottalexander requested a review from a team as a code owner September 3, 2021 18:53
@unclerom
Copy link
Contributor

unclerom commented Sep 6, 2021

This prop is removed in #44

@bootsboogie
Copy link
Contributor

Closing this pr because it ended up getting fixed in #44 which was merged yesterday. Thanks for contributing.

@bootsboogie bootsboogie closed this Sep 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants