Skip to content
This repository has been archived by the owner on Sep 4, 2020. It is now read-only.

Fix INSTALLATION.md docs for v2.0.0 and FirebaseMessaging #1741

Merged
merged 1 commit into from
May 26, 2017
Merged

Fix INSTALLATION.md docs for v2.0.0 and FirebaseMessaging #1741

merged 1 commit into from
May 26, 2017

Conversation

shazron
Copy link
Member

@shazron shazron commented May 25, 2017

Description

The merge clobbered some docs, fixed them.

@@ -267,7 +267,7 @@ To install CocoaPods, please follow the installation instructions [here](https:/
This will clone the required CocoaPods specs-repo into your home folder at `~/.cocoapods/repos`, so it might take a while. See the [CocoaPod Disk Space](#cocoapod-disk-space) section below for more information.


Version `1.9.0` (and above) of this plugin supports [CocoaPods](https://cocoapods.org) installation of the [Google Cloud Messaging](https://cocoapods.org/pods/GoogleCloudMessaging) library.
Version `2.0.0` (and above) of this plugin supports [CocoaPods](https://cocoapods.org) installation of the [Google Firebase Cloud Messaging](https://cocoapods.org/pods/FirebaseMessaging) library.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Just "Firebase Cloud Messaging", I don't think they name it "Google" first

@shazron
Copy link
Member Author

shazron commented May 25, 2017

@macdonst test failure: Object.assign is ES6, and we are still testing this on Travis with Node 0.10. Update to Node 4 for testing?

@macdonst macdonst merged commit 32ff975 into phonegap:master May 26, 2017
@macdonst
Copy link
Member

@shazron yeah, I forgot to do that. Sent a commit to do that right now.

@shazron shazron deleted the cocoapods-doc-update-master branch May 26, 2017 00:19
@lock
Copy link

lock bot commented Jun 3, 2018

This thread has been automatically locked.

@lock lock bot locked as resolved and limited conversation to collaborators Jun 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants