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

Appstore binary rejected for use of private API "transform:" #106

Open
cryptoethic opened this issue Jul 13, 2021 · 3 comments
Open

Appstore binary rejected for use of private API "transform:" #106

cryptoethic opened this issue Jul 13, 2021 · 3 comments

Comments

@cryptoethic
Copy link

cryptoethic commented Jul 13, 2021

Hi,

Working on cordova ios 6.1 with latest version of cordova-plugin-push our binary is uploaded with warnings as described in relating issues:

firebase/firebase-ios-sdk#7177
google/GoogleDataTransport#10

Should this plugin be somehow updated to use newer version of firebase-ios-sdk or i can somehow force it to use newer version?

Maybe this PR might be helpful? #100

All best,
cryptoethic

@cryptoethic
Copy link
Author

Current workaround:

cordova plugin add https://github.com/Salesflare/cordova-plugin-push

Version submitted as pull request #100 by AdriVanHoudt works both on Android and iOS
(FCM on Android)
(APN on iOS)

Looking for any maintainer to check out his solution.

@Heshyo
Copy link

Heshyo commented Sep 15, 2021

Is your app rejected, or do you just get a warning? When a published an update for an app several months ago, I did get a warning but could still publish the update.

Concerning #100, have you tried FCM on iOS?

@cryptoethic
Copy link
Author

@Heshyo It was only warning and we haven't tried to publish an app with that warning.
We are using FCM on Android and APN on iOS. We haven't tried FCM on iOS.

shankari added a commit to shankari/e-mission-phone that referenced this issue Mar 14, 2023
Bump up the minSDK to API 23 (Marshmallow, Android 6)
    - I am not sure why this was API 22; we are certainly not testing on that version

Bump the targetSDK to API 32, android 12 (required for Google Play)
Switch the push and local notification plugins to ones that support the new trampoline functionality
https://developer.android.com/about/versions/12/behavior-changes-12#notification-trampolines

The push plugin is a "dev" version but seems to have all android fixes
It also bumps up the iOS version so that we can avoid the "non-public selectors" error
havesource/cordova-plugin-push#106
e-mission/e-mission-docs#838 (comment)

The push plugin had kotlin build errors that were fixed by adding the gradle preferences
havesource/cordova-plugin-push#218 (comment)

The local notification is a new fork maintained by a new author
which requires us to add the new `AndroidLaunchMode` preference.

The new plugins already have the compile -> implementation fix
So removing the hook added in 6de1f4c

Testing done:
- builds
- generated a local notification by turning off location permissions
- clicking on the notification opened the status screen
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

No branches or pull requests

2 participants