-
Notifications
You must be signed in to change notification settings - Fork 33
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
[iOS] Support open with #1196
Comments
@EtienneLem Thanks for the info in the issue there.
Which I think I had with the original plugin as well :-( |
@EtienneLem any updates on this? I'm not sure what I'm doing wrong... :-( |
This is maybe fixed with missive/cordova-plugin-openwith@17008a8. |
@EtienneLem Thanks for the update! I'll try to use the latest version you mentioned in a few days and report back here what had changed and if this was successful or not. |
@EtienneLem I have created another build with the latest changes in tour branch adn now I'm getting the following error:
Build can be found here: The relevant commit can be seen above and is only changing the package.lock file obviously. |
Right. Cordova really isn’t friendly with multi-targets. We are actually building the iOS app with Fastlane and can set the proper provisioning profiles that way. So we use |
@EtienneLem I was hoping this is handled by the plugin code as well :-( |
Fastlane on windows: |
@BenjaminPoncet I've took the latest code of the fovea open with plugin and I'm still getting the following error:
Any chance you know how to overcome this? |
…ded provisioning profile to build process
@yuanjilee I saw a post on fastlane issue with what seems to be the same issue I'm facing. How did you manage to solve the problem? Are you using two provisioning profiles one for the app and the second one for the share extention? When I use two profiles I get a basically the problem described here: |
* Improve publishing mechanism using faslane #1266, [iOS] Support open with #1196 * [iOS] Support open with #1196 - Fix missing copy using an IHM fork * [iOS] Support open with #1196 - Fix typo in provisioning file * [iOS] Support open with #1196 - a different provisioning profile * [iOS] Support open with #1196 - fix provisioning profile group id, added provisioning profile to build process * [iOS] Support open with #1196 - change provisioning profile to be the same for both targets * [iOS] Support open with #1196 - Fixed plugin code identity and style, changed provisioning profile back * [iOS] Support open with #1196 - Fix quotes in package.json and plugin * [iOS] Support open with #1196 - use same provisioning profile again... * [iOS] Support open with #1196 - show options file content * [iOS] Support open with #1196 - Updated some plugins and ios build tools * [iOS] Support open with #1196 - move cat command to the right place * [iOS] Support open with #1196 - Moved ios configuration to json file * [iOS] Support open with #1196 - fix typo in build-ios file * [iOS] Support open with #1196 - Fix issue with multipe provisioning profile in cordova-ios * [iOS] Support open with #1196 - Added GPX file UTI and added subquery, removed fastlane * [iOS] Support open with #1196 - Added cat to debug plist file * [iOS] Support open with #1196 - changed to wild card plist * [iOS] Support open with #1196 - fix cat call * Restore web and android configurations, changed merged PR packages. * Remove unwanted prints. * Update certificate and provisioning profiles * Fix provisioning profile mess * #1266 - Fix provisioning profile, reduce build matrix * #1266 - Fix provisioning profile uuid * #1266 - Revert matrix build changes
Ok, so an initial commit was merged to master and now the plugin is used in the iOS build in my CI and it completes successfully. yay! |
According to a report from our Facebook group it seems that the app is now listed in some share options but does not open the file when selected. |
https://stackoverflow.com/questions/2774343/how-do-i-associate-file-types-with-an-iphone-application |
This might be relevant for this issue: |
Thanks for the info! Also I've talked with the guys that sent the following PR which solves some of the issues I've listed above: |
Feature
Currently only Android supports open-with/share with due to CI limitation of the plugin that I have chose to use:
j3k0/cordova-plugin-openwith#82
So I'm adding this plugin only when building Android.
I have looked for an iOS only alternative and it seems that the other plugin has this issue too:
EternallLight/cordova-plugin-openwith-ios#23
And so I have written some comments and found out there's another fork that should work for iOS and Android and CI:
https://github.com/missive/cordova-plugin-openwith
I'll need to give it a go.
A few users have already encountered issues with the fact that this is not part of the app, which raises the priority of this issue.
The text was updated successfully, but these errors were encountered: