Page MenuHomePhabricator
Differential D10663

[native][web] Add a context provider on native that exposes notifications session creator
ClosedPublic
Actions

Authored by tomek on Jan 17 2024, 7:59 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Mar 18, 9:24 AM
Unknown Object (File)
Sat, Mar 15, 11:53 PM
Unknown Object (File)
Fri, Mar 14, 9:38 AM
Unknown Object (File)
Mon, Mar 3, 12:47 AM
Unknown Object (File)
Sat, Feb 22, 1:09 PM
Unknown Object (File)
Sat, Feb 22, 1:09 PM
Unknown Object (File)
Sat, Feb 22, 1:09 PM
Unknown Object (File)
Sat, Feb 22, 1:09 PM
View All Files
Subscribers
ashoat

Details

Reviewers
varun
marcin
inka
kamil
Commits
rCOMMbf1ff1e5e8ff: [native][web] Add a context provider on native that exposes notifications…
Summary

This makes it more convenient to use this function on both platforms.

Depends on D10662

Test Plan

Check if encrypted notifs work on iOS and web.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

tomek created this revision.Jan 17 2024, 7:59 AM
tomek held this revision as a draft.
Herald added a subscriber: ashoat. · View Herald TranscriptJan 17 2024, 7:59 AM
tomek edited the summary of this revision. (Show Details)Jan 17 2024, 8:00 AM
tomek added a parent revision: D10662: [web] Introduce deviceID parameter to initial notif message function.
tomek added a child revision: D10664: [native][web] Make identity client mandatory.Jan 17 2024, 8:03 AM
Harbormaster completed remote builds in B25836: Diff 35725.Jan 17 2024, 8:19 AM
tomek published this revision for review.Jan 17 2024, 8:51 AM
tomek retitled this revision from [native][web] Add a context provider on web that exposes notifications session creator to [native][web] Add a context provider on native that exposes notifications session creator.Jan 17 2024, 9:04 AM
kamil accepted this revision.Jan 18 2024, 9:50 AM
kamil added inline comments.
web/root.js
47

the previous order was more readable I think - don't see any dependency why to force IdentityServiceContextProvider to be nested but leaving this up to you

This revision is now accepted and ready to land.Jan 18 2024, 9:50 AM
tomek updated this revision to Diff 35810.Jan 18 2024, 10:12 AM

Reverse providers order

web/root.js
47

Right, that makes sense

Harbormaster completed remote builds in B25909: Diff 35810.Jan 18 2024, 10:32 AM
marcin accepted this revision.Jan 19 2024, 3:07 AM
tomek updated this revision to Diff 35873.Jan 19 2024, 6:20 AM

Rename

Harbormaster completed remote builds in B25951: Diff 35873.Jan 19 2024, 6:38 AM
Closed by commit rCOMMbf1ff1e5e8ff: [native][web] Add a context provider on native that exposes notifications… (authored by tomek). · Explain WhyJan 19 2024, 8:03 AM
This revision was automatically updated to reflect the committed changes.
tomek added a commit: rCOMMbf1ff1e5e8ff: [native][web] Add a context provider on native that exposes notifications….

Revision Contents
Changeset List

PathSize
lib/
shared/
20 lines
types/
10 lines
native/
account/
41 lines
115 lines
web/
account/
15 lines
10 lines

Diff 35725

View Options

lib/shared/notifications-session-creator-context.js

require File.join(File.dirname(`node --print "require.resolve('expo/package.json')"`), "scripts/autolinking") require File.join(File.dirname(`node --print "require.resolve('expo/package.json')"`), "scripts/autolinking")
require File.join(File.dirname(`node --print "require.resolve('react-native/package.json')"`), "scripts/react_native_pods") require File.join(File.dirname(`node --print "require.resolve('react-native/package.json')"`), "scripts/react_native_pods")
require File.join(File.dirname(`node --print "require.resolve('@react-native-community/cli-platform-ios/package.json')"`), "native_modules") require File.join(File.dirname(`node --print "require.resolve('@react-native-community/cli-platform-ios/package.json')"`), "native_modules")
platform :ios, '13.0' platform :ios, '13.0'
install! 'cocoapods', :deterministic_uuids => false install! 'cocoapods', :deterministic_uuids => false
require 'json' require 'json'
def common_comm_target_pods def common_comm_target_pods
pod 'SQLCipher-Amalgamation', :path => '../../node_modules/@commapp/sqlcipher-amalgamation' pod 'SQLCipher-Amalgamation', :path => '../../node_modules/@commapp/sqlcipher-amalgamation'
pod 'react-native-video/VideoCaching', :podspec => '../node_modules/react-native-video' pod 'react-native-video/VideoCaching', :podspec => '../node_modules/react-native-video'
pod 'ffmpeg-kit-react-native', :subspecs => ['min'], :podspec => '../node_modules/ffmpeg-kit-react-native'
end end
post_integrate do |installer| post_integrate do |installer|
begin begin
expo_patch_react_imports!(installer) expo_patch_react_imports!(installer)
rescue => e rescue => e
Pod::UI.warn e Pod::UI.warn e
end end
▲ Show 20 Lines • Show All 81 Lines • Show Last 20 Lines
View Options

lib/types/crypto-types.js

require File.join(File.dirname(`node --print "require.resolve('expo/package.json')"`), "scripts/autolinking") require File.join(File.dirname(`node --print "require.resolve('expo/package.json')"`), "scripts/autolinking")
require File.join(File.dirname(`node --print "require.resolve('react-native/package.json')"`), "scripts/react_native_pods") require File.join(File.dirname(`node --print "require.resolve('react-native/package.json')"`), "scripts/react_native_pods")
require File.join(File.dirname(`node --print "require.resolve('@react-native-community/cli-platform-ios/package.json')"`), "native_modules") require File.join(File.dirname(`node --print "require.resolve('@react-native-community/cli-platform-ios/package.json')"`), "native_modules")
platform :ios, '13.0' platform :ios, '13.0'
install! 'cocoapods', :deterministic_uuids => false install! 'cocoapods', :deterministic_uuids => false
require 'json' require 'json'
def common_comm_target_pods def common_comm_target_pods
pod 'SQLCipher-Amalgamation', :path => '../../node_modules/@commapp/sqlcipher-amalgamation' pod 'SQLCipher-Amalgamation', :path => '../../node_modules/@commapp/sqlcipher-amalgamation'
pod 'react-native-video/VideoCaching', :podspec => '../node_modules/react-native-video' pod 'react-native-video/VideoCaching', :podspec => '../node_modules/react-native-video'
pod 'ffmpeg-kit-react-native', :subspecs => ['min'], :podspec => '../node_modules/ffmpeg-kit-react-native'
end end
post_integrate do |installer| post_integrate do |installer|
begin begin
expo_patch_react_imports!(installer) expo_patch_react_imports!(installer)
rescue => e rescue => e
Pod::UI.warn e Pod::UI.warn e
end end
▲ Show 20 Lines • Show All 81 Lines • Show Last 20 Lines
View Options

native/account/account-hooks.js

require File.join(File.dirname(`node --print "require.resolve('expo/package.json')"`), "scripts/autolinking") require File.join(File.dirname(`node --print "require.resolve('expo/package.json')"`), "scripts/autolinking")
require File.join(File.dirname(`node --print "require.resolve('react-native/package.json')"`), "scripts/react_native_pods") require File.join(File.dirname(`node --print "require.resolve('react-native/package.json')"`), "scripts/react_native_pods")
require File.join(File.dirname(`node --print "require.resolve('@react-native-community/cli-platform-ios/package.json')"`), "native_modules") require File.join(File.dirname(`node --print "require.resolve('@react-native-community/cli-platform-ios/package.json')"`), "native_modules")
platform :ios, '13.0' platform :ios, '13.0'
install! 'cocoapods', :deterministic_uuids => false install! 'cocoapods', :deterministic_uuids => false
require 'json' require 'json'
def common_comm_target_pods def common_comm_target_pods
pod 'SQLCipher-Amalgamation', :path => '../../node_modules/@commapp/sqlcipher-amalgamation' pod 'SQLCipher-Amalgamation', :path => '../../node_modules/@commapp/sqlcipher-amalgamation'
pod 'react-native-video/VideoCaching', :podspec => '../node_modules/react-native-video' pod 'react-native-video/VideoCaching', :podspec => '../node_modules/react-native-video'
pod 'ffmpeg-kit-react-native', :subspecs => ['min'], :podspec => '../node_modules/ffmpeg-kit-react-native'
end end
post_integrate do |installer| post_integrate do |installer|
begin begin
expo_patch_react_imports!(installer) expo_patch_react_imports!(installer)
rescue => e rescue => e
Pod::UI.warn e Pod::UI.warn e
end end
▲ Show 20 Lines • Show All 81 Lines • Show Last 20 Lines
View Options

native/root.react.js

require File.join(File.dirname(`node --print "require.resolve('expo/package.json')"`), "scripts/autolinking") require File.join(File.dirname(`node --print "require.resolve('expo/package.json')"`), "scripts/autolinking")
require File.join(File.dirname(`node --print "require.resolve('react-native/package.json')"`), "scripts/react_native_pods") require File.join(File.dirname(`node --print "require.resolve('react-native/package.json')"`), "scripts/react_native_pods")
require File.join(File.dirname(`node --print "require.resolve('@react-native-community/cli-platform-ios/package.json')"`), "native_modules") require File.join(File.dirname(`node --print "require.resolve('@react-native-community/cli-platform-ios/package.json')"`), "native_modules")
platform :ios, '13.0' platform :ios, '13.0'
install! 'cocoapods', :deterministic_uuids => false install! 'cocoapods', :deterministic_uuids => false
require 'json' require 'json'
def common_comm_target_pods def common_comm_target_pods
pod 'SQLCipher-Amalgamation', :path => '../../node_modules/@commapp/sqlcipher-amalgamation' pod 'SQLCipher-Amalgamation', :path => '../../node_modules/@commapp/sqlcipher-amalgamation'
pod 'react-native-video/VideoCaching', :podspec => '../node_modules/react-native-video' pod 'react-native-video/VideoCaching', :podspec => '../node_modules/react-native-video'
pod 'ffmpeg-kit-react-native', :subspecs => ['min'], :podspec => '../node_modules/ffmpeg-kit-react-native'
end end
post_integrate do |installer| post_integrate do |installer|
begin begin
expo_patch_react_imports!(installer) expo_patch_react_imports!(installer)
rescue => e rescue => e
Pod::UI.warn e Pod::UI.warn e
end end
▲ Show 20 Lines • Show All 81 Lines • Show Last 20 Lines
View Options

web/account/account-hooks.js

Loading...
View Options

web/root.js

Loading...