-
Notifications
You must be signed in to change notification settings - Fork 223
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
Add example App #206
Add example App #206
Conversation
6deab26
to
66ed7dc
Compare
@luacmartins this is the PR that introduces Example app, it's mostly commits moved from our old fork, with some small tweaks so it works and builds correctly. CC @grgia @awesomejerry (I have no idea if you're still contributing to this or not - feel free to comment if you do) |
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.
Just some comments. Looks good overall 🚀
"@svgr/webpack": "^8.1.0", | ||
"file-loader": "^6.2.0", | ||
"react": "18.2.0", | ||
"react-native": "^0.73.6", |
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.
Would be nice to bump the version to newest RN in some follow-ups. But then macos
would probably stop working so idk.
d2e5e37
to
1dc3fea
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.
LGTM
Description
This PR adds an example app to help developers who want to work locally on
react-native-qrcode-svg
.All of the important code is kept within
Example
directory. This PR is expected to introduce no changes to the library or api, it just adds useful developer toolingrec-qrcode.mov
Details
yarn
because when I usednpm
the locallink:...
dependency doesn't work; I have not found a better way to link to local library that wouldn't break alsoSupported platforms:
Important files: