Skip to content
This repository was archived by the owner on Apr 8, 2025. It is now read-only.

Conversation

TrekSoft
Copy link

The getPackages method has changed with the latest version of React Native to not include the default firebase package and requires a different way of adding module packages. This updates the documentation for Auth to reflect this. If this PR is approved, I can make this same change across other modules.

The getPackages method has changed with the latest version of React Native to not include the default firebase package and requires a different way of adding module packages. This updates the documentation for Auth to reflect this. If this PR is approved, I can make this same change across other modules.
@CLAassistant
Copy link

CLAassistant commented Aug 17, 2019

CLA assistant check
All committers have signed the CLA.

Copy link
Collaborator

@mikehardy mikehardy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this will be hugely helpful to people in the future - RN60 is just now starting to be adopted more widely I think

@mikehardy
Copy link
Collaborator

almost perfect :-), then can be clone-stamped photoshop style to the others

@TrekSoft
Copy link
Author

Done!

@invertase invertase deleted a comment from TrekSoft Aug 18, 2019
Copy link
Collaborator

@mikehardy mikehardy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks awesome - thank you for cleaning this up for RN60!

@mikehardy mikehardy merged commit aadd168 into invertase:v5.x.x Aug 18, 2019
@mikehardy
Copy link
Collaborator

should be live now!

@TrekSoft
Copy link
Author

Awesome! Glad to contribute since this project (and excellent documentation) has been such a help to me!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants