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.

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

change hook triggering iosAddTarget.js. #72

Merged
merged 1 commit into from
Jan 1, 2020
Merged

Conversation

devtobo
Copy link
Contributor

@devtobo devtobo commented Sep 18, 2019

Problem: If the plugin is already installed, iosAddTarget.js will not be triggered. You have to remove and re add the plugin each times.

These issues are related:
#60
EternallLight/cordova-plugin-openwith-ios#19

Solution, change hook from after_plugin_install to after_plugin_add which is triggered even if the plugin is already installed

Copy link
Collaborator

@NiklasMerz NiklasMerz left a comment

Choose a reason for hiding this comment

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

Changing to this lifecyle is indeed good. I tried this in a fork as well.

JaimeFerBec pushed a commit to JaimeFerBec/cordova-plugin-openwith that referenced this pull request Oct 10, 2019
@j3k0 j3k0 mentioned this pull request Jan 1, 2020
@NiklasMerz NiklasMerz merged commit 7920363 into j3k0:master Jan 1, 2020
BenjaminPoncet pushed a commit to BenjaminPoncet/cordova-plugin-openwith that referenced this pull request Mar 4, 2020
BenjaminPoncet added a commit to BenjaminPoncet/cordova-plugin-openwith that referenced this pull request Mar 4, 2020
…being added to xcode project. (j3k0#72)"

This reverts commit e87572e.
dnlek added a commit to dnlek/cordova-plugin-openwith that referenced this pull request Jul 22, 2021
 into j3k0-master

* 'master' of https://github.com/j3k0/cordova-plugin-openwith:
  Fix missing quotes
  Change code sign indentity and style
  Fix async call
  Added a copy files to after install hook
  IOS : Fix __BUNDLE_IDENTIFIER__ to prevent double suffix
  IOS : Add ShareExtension-Entitlements.plist to package.json
  IOS : Remove "iOS Setup" section
  IOS : Fix App Group creation and fix Share Extension Bundle Identifier
  IOS : Fix getCordovaParameter() to use package.json Because after_prepare hook does not pass installation variables in process.argv
  IOS : Move hooks on before and after prepare This allows ShareExtension to be created and updated in any case
  2.0.0
  Bump version 2.0.0
  change hook triggering iosAddTarget.js. Fixes the target not being added to xcode project. (j3k0#72)
  Fix iOS build flags (j3k0#84)
  Fix extension crash on iOS 13 (j3k0#69)
  1.3.0
  Set version in plugin.xml
dnlek added a commit to dnlek/cordova-plugin-openwith that referenced this pull request Jul 22, 2021
* j3k0-master:
  Fix missing quotes
  Change code sign indentity and style
  Fix async call
  Added a copy files to after install hook
  IOS : Fix __BUNDLE_IDENTIFIER__ to prevent double suffix
  IOS : Add ShareExtension-Entitlements.plist to package.json
  IOS : Remove "iOS Setup" section
  IOS : Fix App Group creation and fix Share Extension Bundle Identifier
  IOS : Fix getCordovaParameter() to use package.json Because after_prepare hook does not pass installation variables in process.argv
  IOS : Move hooks on before and after prepare This allows ShareExtension to be created and updated in any case
  2.0.0
  Bump version 2.0.0
  change hook triggering iosAddTarget.js. Fixes the target not being added to xcode project. (j3k0#72)
  Fix iOS build flags (j3k0#84)
  Fix extension crash on iOS 13 (j3k0#69)
  1.3.0
  Set version in plugin.xml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants