Skip to content

Conversation

@onys-programmer
Copy link
Contributor

@onys-programmer onys-programmer commented Aug 19, 2023

Motivation

Fixes # (issue)

Type of change

  • This change requires a documentation update

How Has This Been Tested?

Checklist

  • My code follows the style guidelines of this project
  • My changes generate no new warnings

Additional context

image

checkedIcon and uncheckedIcon props need to be updated
because of below error
Warning: Failed prop type: Invalid prop `name` of value `circle-o` supplied to `Icon`
@netlify
Copy link

netlify bot commented Aug 19, 2023

Deploy Preview for react-native-elements ready!

Name Link
🔨 Latest commit c559fd5
🔍 Latest deploy log https://app.netlify.com/sites/react-native-elements/deploys/64fc405fee1d130008b6ee6d
😎 Deploy Preview https://deploy-preview-3826--react-native-elements.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@codecov
Copy link

codecov bot commented Sep 9, 2023

Codecov Report

Merging #3826 (3c1a72a) into next (a47e2f0) will not change coverage.
The diff coverage is n/a.

❗ Current head 3c1a72a differs from pull request most recent head c559fd5. Consider uploading reports for the commit c559fd5 to get more accurate results

@@           Coverage Diff           @@
##             next    #3826   +/-   ##
=======================================
  Coverage   79.69%   79.69%           
=======================================
  Files          87       87           
  Lines        1842     1842           
  Branches      811      811           
=======================================
  Hits         1468     1468           
  Misses        369      369           
  Partials        5        5           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@arpitBhalla arpitBhalla merged commit 34d2cd7 into react-native-elements:next Sep 9, 2023
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.

2 participants