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.

Sign up for GitHub

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

Jump to bottom

[WIP] v3.0.0 #314

Closed
wants to merge 45 commits into from
Closed

[WIP] v3.0.0 #314

wants to merge 45 commits into from

Conversation

Salakar
Copy link
Contributor

@Salakar Salakar commented Aug 7, 2017 •
edited
Loading

Salakar added 30 commits June 29, 2017 17:24
@Salakar
started work on multiple app initialization
80ae842
@Salakar
more work on multiple app initialization
ec22e51
@Salakar
more work on multiple apps support
f97db8e
@Salakar
Merge branch 'master' of https://github.com/invertase/react-native-fi…
c6773dc 
…rebase into multi-apps

# Conflicts:
#	android/src/main/java/io/invertase/firebase/auth/RNFirebaseAuth.java
@Salakar
[android][database] switch to SparseArray for database references
82a2afd
@Salakar
[both] continued work on multi app support - storage now full support…
282f953 
…ed and auth WIP
@Salakar
Merge branch 'master' of https://github.com/invertase/react-native-fi…
fc84452 
…rebase into multi-apps
@Salakar
[both] re-enabled analytics, admob, crash & perf modules
91b7281
@Salakar
[android][config] removed appName requirement on enableDeveloperMode …
b9b9de5 
…- native SDK's don't support multiple apps for remote config
@Salakar
Merge branch 'master' of https://github.com/invertase/react-native-fi…
ef259fb 
…rebase into multi-apps
@Salakar
[android][app] apps initialized natively now automatically initialize…
9a3e224 
…d js side - via react module getConstants
@Salakar
[ios][app] apps initialized natively now automatically initialized js…
813b91f 
… side - via react module getConstants
@Salakar
[android][auth] fix unlink method - missing appName / targeted app in…
a0c42c7 
…stance after master merge
@Salakar
[ios][auth] fix unlink method - missing appName / targeted app instan…
48570f5 
…ce after master merge
@Salakar
[docs] start of app/core documentation
471539a
@Salakar
[android][ios][core] initialize app parameters no longer prefixed by …
075cd69 
…platform name i.e iosClientId becomes clientId. Use react native platform module to provide platform specific options.
@Salakar
[tests] update firebase helper to initialize additional apps
2d744b1
@Salakar
[auth][ios] finalised auth changed listener logic
d8fee7f
@Salakar
[tests] improved testing framework error/success output
c4d066f
@Salakar
[auth][android] remove firebase providerData from user responses, s…
e96c5db 
…ee: firebase/firebase-ios-sdk#140
@Salakar
[core][both] added delete app method (android is just a stub method a…
0337725 
…s the firebase sdk does not support it)
@Salakar
[messaging][js] fixed EE issue
294ee00
@Salakar
[core][js] misc cleanup/validation
267f908
@Salakar
[tests] removed misc logging
9b5e361
@Salakar
[tests][core] start of core/app tests
d64e549
@Salakar
Merge branch 'master' of https://github.com/invertase/react-native-fi…
3347041 
…rebase into multi-apps
@Salakar
Merge branch 'master' of https://github.com/invertase/react-native-fi…
9c90939 
…rebase into multi-apps
@Salakar
Merge branch 'master' of https://github.com/invertase/react-native-fi…
6b7647c 
…rebase into multi-apps
@Salakar
[database][wip] refactor & improvements to add support for multiple apps
e3d1261
@Salakar
Merge branch 'master' of https://github.com/invertase/react-native-fi…
7fc40a2 
…rebase into multi-apps
Salakar and others added 12 commits July 31, 2017 18:25
@Salakar
[database][wip] update reference to support 'ThenableReference'
c9efb00
@Salakar
[database][wip] misc multi-app
b935034
@Salakar
Merge branch 'master' of https://github.com/invertase/react-native-fi…
3504715 
…rebase into multi-apps
@chrisbianca
[database][ios][wip] Replicate Android changes on iOS
a785b05
@Salakar
Merge branch 'master' of https://github.com/invertase/react-native-fi…
6298b86 
…rebase into multi-apps

# Conflicts:
#	ios/RNFirebase.xcodeproj/project.pbxproj
@Salakar
[android][database] updated keepSynced to now work with modifiers
913bb4e
@Salakar
[ios][database] updated keepSynced to now work with modifiers
78e9f4b
@Salakar
[js][database] fixed an issue where query methods were not returning …
d2909be 
…correctly
@Salakar
[js][database] remove, update, setWithPriority, setPriority & set now…
53acfe3 
… correctly support callbacks or promises

[js][database] fixed query modifier methods - were accessing incorrect query property
@Salakar
[tests][database] fixed broken 'root' tests - were incorrectly access…
bec4e8c 
…ing a private property - switched to using 'path'
@Salakar
[js][database] cleanup transaction completion wrapper
fb6251a
@Salakar
[js][database] once now supports passing context - as per the web sdk
425dfbc
@Salakar Salakar added this to the v3.0.0 milestone Aug 7, 2017
@Salakar Salakar self-assigned this Aug 7, 2017
@Salakar Salakar added plugin: analytics Google Analytics for Firebase platform: android plugin: authentication Firebase Authentication plugin: app-core Firebase Apps / Core internals. plugin: database Firebase Realtime Database platform: ios platform: javascript plugin: messaging FCM only - ( messaging() ) - do not use for Notifications plugin: storage Firebase Cloud Storage labels Aug 7, 2017
@Salakar Salakar closed this Aug 15, 2017
@Salakar Salakar deleted the multi-apps branch August 26, 2017 01:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Salakar Salakar

Labels
platform: android platform: ios platform: javascript plugin: analytics Google Analytics for Firebase plugin: app-core Firebase Apps / Core internals. plugin: authentication Firebase Authentication plugin: database Firebase Realtime Database plugin: messaging FCM only - ( messaging() ) - do not use for Notifications plugin: storage Firebase Cloud Storage
Projects
None yet
Milestone
v3.0.0 RC1
Development

Successfully merging this pull request may close these issues.
3 participants
@Salakar @Ehesp @chrisbianca