Skip to content
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

Changing library from ART to react-native-svg #50

Closed
wants to merge 4 commits into from

Conversation

cdesch
Copy link

@cdesch cdesch commented Dec 19, 2019

This change removes ART and adds react-native-svg as the drawing library for producing the barcode. This change is to support the deprecation of ART from the ReactNative libraries. The react community ART library is not linked in Expo v36 projects. SVG is the preferred method. This change is linked to issue #49 #49

There are no tests for this changes. Changes will need to be made to the readme first.

This change removes ART and adds react-native-svg as the drawing library for producing the barcode. This change is to support the deprecation of ART from the ReactNative libraries. The react community ART library is not linked in Expo v36 projects. SVG is the preferred method. This change is linked to issue wonsikin#49 wonsikin#49
@wonsikin
Copy link
Owner

Thanks for your contribution. But we decide to use @react-native-community/art

@wonsikin wonsikin closed this Mar 29, 2020
@formaldehydeson
Copy link

thanks for this, only way I could get it working on iOS using Expo

@donni106
Copy link

thanks for this, only way I could get it working on iOS using Expo

same for me. copied @cdesch code to my project and use that instead of the package, which results in: #59

@JonathanMai
Copy link

same here, not using expo and having the same issue.
Thanks @cdesch for the fix! :)

@namti
Copy link

namti commented Jan 12, 2024

Switching to https://www.npmjs.com/package/expo-barcode-generator solves this issue.

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.

6 participants