-
Notifications
You must be signed in to change notification settings - Fork 240
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
Does react-native-todo-list
need updating?
#1499
Labels
Comments
doelgonzo
added
bug
Something isn't working
triage needed
The issue needs to be reviewed by the team
labels
Feb 29, 2024
adigau
removed
triage needed
The issue needs to be reviewed by the team
public repo
labels
Apr 15, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug
Checking out the react-native example, I tried pulling it down on my Mac but my mac version required I use iOS 12+, so I simply copied the config and modeled my app to follow the patterns in the example. However, I ran into errors around
window.postMessage
andwindow.addEventListener
being undefined. I mocked those, and it seems like the errors no longer happen, but (1) the docs don't capture that, and (2) It's unclear if setting this will affect functionality at all (even though it seems like the example works.Now, my example uses the latest react-native version (0.73.5), so I don't know if that has anything to do with needing to stub out those window functions, as the example doesn't stub them out. Should this be the new way of doing it?
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Example repo should serve as an example for usage in react-native environment.
Illustrations
N/A
Environment (please complete the following information):
Additional context
N/A
The text was updated successfully, but these errors were encountered: