This repository has been archived by the owner on Sep 4, 2020. It is now read-only.
Remove unused files #1732
Merged
Remove unused files #1732
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Thanks @jcesarmobile I actually planned on doing this after commit 46ba274 |
macdonst
pushed a commit
that referenced
this pull request
May 24, 2017
macdonst
added a commit
that referenced
this pull request
May 25, 2017
* Added partial Android FCM support (#975) * Modified plugin.xml to include FCM changes * Plugin.xml changes for FCM * Java changes for FCM * Increased plugin version to 2.0.0 * Increased plugin version in package.json * Added topic subscription/unsubscription * Removed some empty lines * Fixed empty token on android first run (#1008) * Removed GCM files and added FCM files * Added .framework files as custom frameworks * Changed code to work with FCM * Added hook and resource file to copy GoogleService-Info.plist * fixed registration and removed unused code * Use CocoaPods * Fix rebase errors * Fix topic subscription and unsubscription on FCM * Refactor GCMIntentService to FCMService * Issue #689: Remove sender id from PushNotification init Android options * Issue #689: Remove sender id from PushNotification init iOS options * 📝 Update to using fcm-node * 🐛 Issue #1188: Strings.xml google_app_id conflict with google-services.json * Add empty google services plist * Pin FCM to 9.8.0 * 🔧 Add browser platform back for FCM branch * 🔨🔧⬆️ Use Babel to transpile ES2015 code * 🔧 add tern to gitignore * 🔖 Bumping plugin version to 2.0.0-rc1 * 2.0.0-rc1 * Updating CHANGELOG * Fixing my merge error for PR #1378 Optional event emit instead of function call for action buttons * 🐛🍎 Issue #1461: App crashes when initialising with topics (v2.0.0-rc1) * 🐛🐧📝⬆️ #1460: Build Issue Android * 🐛🍎🔧 Fixing a merge issue where aps-environment was accidentally removed * 🐧📝🐛 Issue #1470: Cannot install phonegap-plugin-push on master * 📝 Using a newer version on the examples * 🍎🐛 Issue #1497: App crashes after refreshing when using FCM (v2) * Bumping plugin version to 2.0.0-rc2 * 2.0.0-rc2 * Updating CHANGELOG * Added partial Android FCM support (#975) * Modified plugin.xml to include FCM changes * Plugin.xml changes for FCM * Java changes for FCM * Increased plugin version to 2.0.0 * Increased plugin version in package.json * Added topic subscription/unsubscription * Removed some empty lines * Removed GCM files and added FCM files * Changed code to work with FCM * Added hook and resource file to copy GoogleService-Info.plist * Use CocoaPods * Fix rebase errors * Issue #689: Remove sender id from PushNotification init Android options * 🐛 Issue #1188: Strings.xml google_app_id conflict with google-services.json * Pin FCM to 9.8.0 * 🔧 Add browser platform back for FCM branch * 🔨🔧⬆️ Use Babel to transpile ES2015 code * 🔧 add tern to gitignore * 🔖 Bumping plugin version to 2.0.0-rc1 * 🐛🐧📝⬆️ #1460: Build Issue Android * Bumping plugin version to 2.0.0-rc2 * Added partial Android FCM support (#975) * Modified plugin.xml to include FCM changes * Plugin.xml changes for FCM * Java changes for FCM * Increased plugin version to 2.0.0 * Increased plugin version in package.json * Added topic subscription/unsubscription * Removed some empty lines * Removed GCM files and added FCM files * Added hook and resource file to copy GoogleService-Info.plist * Use CocoaPods * Fix rebase errors * 🐛 Issue #1188: Strings.xml google_app_id conflict with google-services.json * Updating CHANGELOG * ✏️ fixing error in CHANGELOG * Bumping plugin version to 2.0.0-rc2 * Updating CHANGELOG * Added partial Android FCM support (#975) * Modified plugin.xml to include FCM changes * Plugin.xml changes for FCM * Java changes for FCM * Increased plugin version to 2.0.0 * Increased plugin version in package.json * Added topic subscription/unsubscription * Removed some empty lines * Removed GCM files and added FCM files * Changed code to work with FCM * Added hook and resource file to copy GoogleService-Info.plist * fixed registration and removed unused code * Use CocoaPods * Fix rebase errors * Issue #689: Remove sender id from PushNotification init Android options * Issue #689: Remove sender id from PushNotification init iOS options * 🐛 Issue #1188: Strings.xml google_app_id conflict with google-services.json * 🔧 add tern to gitignore * 🔖 Bumping plugin version to 2.0.0-rc1 * Bumping plugin version to 2.0.0-rc2 * Remove hook and use resource-file tag to copy google services file * Add resource-file way of copying google services files * Bump requirements * 🔧 Update cordovaDependencies * 🔖 Bumping plugin version to 2.0.0-rc3 * 2.0.0-rc3 * Updating CHANGELOG * Fixes 'framework not found GoogleToolboxForMac' linker error in 2.0.0-rc3 (#1715) * Remove unused files (#1732) * Remove GTM framework. (#1739)
|
This thread has been automatically locked. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
The resource-file tag was used to create the Resources folder with an empty GoogleService-Info.plist file easily so the hook could write on it, but as the hook is removed it doesn't make sense to keep the resource-file tag here anymore.
Also removed the
scripts/copy_file.jsfile (hook) and the empty GoogleService-Info.plistRelated Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: