Page MenuHomePhabricator
Differential D5925

[native] [30/40] RN 0.70: Patch react-native-firebase to avoid Android NativeEventEmitter warning
ClosedPublic
Actions

Authored by ashoat on Dec 18 2022, 8:02 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 7, 3:48 AM
Unknown Object (File)
Sat, Apr 5, 12:25 PM
Unknown Object (File)
Thu, Apr 3, 10:25 PM
Unknown Object (File)
Fri, Mar 28, 8:35 AM
Unknown Object (File)
Sat, Mar 15, 7:05 AM
Unknown Object (File)
Mar 4 2025, 2:36 AM
Unknown Object (File)
Mar 4 2025, 2:36 AM
Unknown Object (File)
Mar 4 2025, 2:36 AM
View All Files
Subscribers
atul

Details

Reviewers
marcin
tomek
bartek
Commits
rCOMMf271afbe4e6e: [native] [30/40] RN 0.70: Patch react-native-firebase to avoid Android…
Summary

Without this change a warning is printed. The methods don't need to be filled in, they just need to be defined. Here's an example of react-native-orientation-locker fixing the same issue.

The right long-term solution is to migrate all of our Android notifs logic to be using our own modules, but in order to unblock the React Native update I'm just patching this now.

Depends on D5924

Test Plan
  1. Confirm Android NativeEventEmitter warning is no longer printed
  2. Tested along with whole stack: test plan (includes testing notifs on Android)

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

ashoat created this revision.Dec 18 2022, 8:02 PM
Herald added a subscriber: atul. · View Herald TranscriptDec 18 2022, 8:02 PM
Harbormaster returned this revision to the author for changes because remote builds failed.Dec 18 2022, 8:03 PM
Harbormaster failed remote builds in B14254: Diff 19493!
ashoat retitled this revision from [native] [29/40] RN 0.70: Patch react-native-firebase to avoid Android NativeEventEmitter warning to [native] [30/40] RN 0.70: Patch react-native-firebase to avoid Android NativeEventEmitter warning.Dec 18 2022, 8:06 PM
ashoat updated this revision to Diff 19514.Dec 18 2022, 8:20 PM

Rebase

ashoat updated this revision to Diff 19541.Dec 18 2022, 8:33 PM

Rebase

Harbormaster failed remote builds in B14275: Diff 19514!Dec 18 2022, 8:34 PM
Harbormaster returned this revision to the author for changes because remote builds failed.Dec 18 2022, 9:37 PM
Harbormaster failed remote builds in B14302: Diff 19541!
ashoat added a child revision: D5932: [native] [31/40] RN 0.70: First commit for new AndroidLifecycle Expo module.Dec 19 2022, 7:00 AM
ashoat updated this revision to Diff 19603.Dec 19 2022, 8:22 AM

Rebase

ashoat updated this revision to Diff 19636.Dec 19 2022, 8:38 AM

Rebase

Harbormaster failed remote builds in B14354: Diff 19603!Dec 19 2022, 9:10 AM
ashoat published this revision for review.Dec 19 2022, 9:11 AM

Please ignore CI until the end of the stack

Harbormaster failed remote builds in B14387: Diff 19636!Dec 19 2022, 10:11 AM
ashoat updated this revision to Diff 19709.Dec 19 2022, 12:30 PM

Rebase

Harbormaster failed remote builds in B14454: Diff 19709!Dec 19 2022, 12:54 PM
ashoat updated this revision to Diff 19767.Dec 20 2022, 6:22 AM

Rebase

Harbormaster failed remote builds in B14510: Diff 19767!Dec 20 2022, 6:27 AM
tomek accepted this revision.Dec 20 2022, 7:12 AM
This revision is now accepted and ready to land.Dec 20 2022, 7:12 AM
bartek accepted this revision.Dec 20 2022, 8:33 AM
ashoat updated this revision to Diff 19829.Dec 20 2022, 11:02 AM

Rebase

Harbormaster completed remote builds in B14569: Diff 19829.Dec 20 2022, 11:40 AM
Closed by commit rCOMMf271afbe4e6e: [native] [30/40] RN 0.70: Patch react-native-firebase to avoid Android… (authored by ashoat). · Explain WhyDec 20 2022, 12:01 PM
This revision was automatically updated to reflect the committed changes.
ashoat added a commit: rCOMMf271afbe4e6e: [native] [30/40] RN 0.70: Patch react-native-firebase to avoid Android….

Revision Contents
Changeset List

PathSize
patches/
38 lines

Diff 19767

View Options

patches/react-native-firebase+5.6.0.patch

Show First 20 Lines • Show All 55 Lines • ▼ Show 20 Lines networks:
- services-net - services-net
build: build:
dockerfile: services/identity/Dockerfile dockerfile: services/identity/Dockerfile
context: ../ context: ../
image: commapp/identity-server:0.1 image: commapp/identity-server:0.1
container_name: identity-server container_name: identity-server
ports: ports:
- "${COMM_SERVICES_PORT_IDENTITY}:50051" - "${COMM_SERVICES_PORT_IDENTITY}:50051"
# localstack
localstack:
image: localstack/localstack
container_name: localstack
hostname: localstack
ports:
- "4566:4566"
environment:
- SERVICES=s3,dynamodb
- DATA_DIR=${TMPDIR}/localstack/data
- HOSTNAME_EXTERNAL=localstack
volumes:
jimUnsubmitted
Not Done
Inline Actions

Why bind mount a temp directory instead of using a data volume? It's just gonna create issues with permissions. Also ${TMPDIR} is not used to refer to a path in both the host and the container which is a bad idea.

jim: Why bind mount a temp directory instead of using a data volume? It's just gonna create issues…
- ${TMPDIR}:/tmp/localstack
networks:
- services-net