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

fix(animated): load NativeAnimatedModule on macOS #2302

Merged
merged 1 commit into from
Dec 12, 2024

Conversation

Saadnajmi
Copy link
Collaborator

Summary:

Resolves #2301

Test Plan:

CI should pass

@tido64
Copy link
Member

tido64 commented Dec 4, 2024

Oh, there's another one in Modal.js:

Platform.OS !== 'ios' ? null : NativeModalManager,

Are you looking into it?

@Saadnajmi
Copy link
Collaborator Author

Oh, there's another one in Modal.js:

Platform.OS !== 'ios' ? null : NativeModalManager,

Are you looking into it?

We don't have Modal on macOS, so I skipped that one (there were a couple of iOS only checks). But maybe the code still runs so let me double check that.

@Saadnajmi Saadnajmi changed the title fix(NativeEventEmitter): load NativeAnimatedModule on macOS fix(animated): load NativeAnimatedModule on macOS Dec 6, 2024
@Saadnajmi Saadnajmi merged commit 16acfc3 into microsoft:main Dec 12, 2024
14 checks passed
@Saadnajmi Saadnajmi deleted the fix-event-emitter branch December 12, 2024 01:51
@gabrieldonadel
Copy link

When can we expect this to be released?

@Saadnajmi
Copy link
Collaborator Author

When can we expect this to be released?

I need to backport to 0.76, but hopefully by Friday

Saadnajmi added a commit to Saadnajmi/react-native-macos that referenced this pull request Dec 12, 2024
Saadnajmi added a commit that referenced this pull request Dec 13, 2024
* fix(NativeEventEmitter): load NativeAnimatedModule on macOS (#2302)

* update lock
@Saadnajmi
Copy link
Collaborator Author

@gabrieldonadel try this with RNM 0.76.6?

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.

NativeAnimatedModule causes "Invariant Violation: new NativeEventEmitter() requires a non-null argument."
3 participants