-
Notifications
You must be signed in to change notification settings - Fork 72
Plain JS ES6 class works with latest RN #17
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
base: master
Are you sure you want to change the base?
Conversation
|
TOP! Actually had to change the styling part, as it was not working correctly with this:
|
|
Hey @alexanderniebuhr what change did you have to make? Can you paste the code snippet here? |
|
To
I did change this two lines back to the old ones, so I could use the fillStyle prop as shown in the readme from the module in my main class component :) There might be an other way to set fillStyle, but I want to set it in my class and not in the node_modules folder |
|
@alexanderniebuhr ah ok good catch! I missed that because I was using default stylings, tried to customize it and I see what you mean. Thanks for the heads up! I'll update the PR 🚀 👍 🌈 |
|
No probs. I think there might be, a way with your option, but for me it was the easiest to change it back :) |
52534b2 to
7f08ab9
Compare
|
@alexanderniebuhr ok I made an update, the style override works for me now You can test it in your project via this if you like 👍 |
|
I'll try soon , and will post feedback here :) |
|
cool! |
|
@alexanderniebuhr what did you import look like when you used the version as: did it look like this? |
|
I used import from the readme of the initial module Von meinem iPhone gesendet
|
7f08ab9 to
3ce389b
Compare
|
@alexanderniebuhr @lwansbrough updated ❤️ |
|
@lwansbrough beep 🚀 :) |

@lwansbrough Updated the component to use plain JS ES6 classes which works with newest versions of
react-native🚀