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

can't update using extension #214

Open
raresn opened this issue Dec 11, 2024 · 13 comments
Open

can't update using extension #214

raresn opened this issue Dec 11, 2024 · 13 comments

Comments

@raresn
Copy link

raresn commented Dec 11, 2024

We recently updated air sdk to the latest version, and trying to update the extensions as well using APM.

We are using:
com.android.installreferrer
com.distriqt.Application
com.distriqt.CameraRollExtended
com.distriqt.Contacts
com.distriqt.firebase.Crashlytics
com.distriqt.firebase.Database
com.distriqt.IDFA
com.distriqt.InAppBilling
com.distriqt.PushNotifications-FCM

We tryed to update / reinstall but we get confilcts on Contacts and PushNotifications-FCM.

CONFLICT :: fatal error : found [13] conflicting packages
CONFLICT :: androidx.appcompat
CONFLICT :: ├── 2.0.0 required by: [email protected]
CONFLICT :: └── 1.6.104 required by: [email protected]
CONFLICT :: androidx.browser
CONFLICT :: ├── 2.0.0 required by: [email protected]
CONFLICT :: └── 1.5.4 required by: [email protected]
CONFLICT :: androidx.core
CONFLICT :: ├── 2.0.0 required by: [email protected]
CONFLICT :: ├── 1.13.100 required by: [email protected]
CONFLICT :: └── 1.10.101 required by: [email protected]
CONFLICT :: androidx.exifinterface
CONFLICT :: ├── 2.0.0 required by: [email protected]
CONFLICT :: └── 1.3.604 required by: [email protected]
CONFLICT :: com.bumptech.glide
CONFLICT :: ├── 5.0.0 required by: [email protected]
CONFLICT :: └── 4.12.7 required by: [email protected]
CONFLICT :: com.distriqt.playservices.Base
CONFLICT :: ├── 19.0.0 required by: [email protected]
CONFLICT :: └── 18.2.4 required by: [email protected]
CONFLICT :: com.distriqt.playservices.CloudMessaging
CONFLICT :: ├── 18.0.0 required by: [email protected]
CONFLICT :: ├── 17.2.0 required by: [email protected]
CONFLICT :: └── 17.0.205 required by: [email protected]
CONFLICT :: com.distriqt.square.okhttp3
CONFLICT :: ├── 5.0.0 required by: [email protected]
CONFLICT :: └── 3.140.900 required by: [email protected]
CONFLICT :: com.google.android.datatransport
CONFLICT :: ├── 4.0.0 required by: [email protected]
CONFLICT :: └── 3.0.7 required by: [email protected]
CONFLICT :: com.google.dagger
CONFLICT :: ├── 3.0.0 required by: [email protected]
CONFLICT :: └── 2.28.304 required by: [email protected]
CONFLICT :: com.google.firebase.core
CONFLICT :: ├── 34.0.0 required by: [email protected]
CONFLICT :: └── 33.0.2 required by: [email protected]
CONFLICT :: com.google.guava
CONFLICT :: ├── 32.0.0 required by: [email protected]
CONFLICT :: └── 31.1.107 required by: [email protected]
CONFLICT :: com.jetbrains.kotlin
CONFLICT :: ├── 2.0.0 required by: [email protected]
CONFLICT :: └── 1.8.220 required by: [email protected]
@marchbold
Copy link
Contributor

Hi, This is stating the latest versions of the extensions aren't compatible. You'll need to install the previous version of Firebase until we have completed updating all the extensions

@raresn
Copy link
Author

raresn commented Dec 12, 2024

Well, i tried to go backwards 1 version, but no luck.

i get this error with 15.0.2

Installed package : [email protected]
Installing package : [email protected]
Downloading package : [email protected]
✗ [404] There was an issue accessing the package
ERROR :: Failed to install package : [email protected]
install :: [404] There was an issue accessing the package

But i also get this error with 15.0.1
Installed package : [email protected]
Installing package : [email protected]
Downloading package : [email protected]
✗ [404] There was an issue accessing the package
ERROR :: Failed to install package : [email protected]
install :: [404] There was an issue accessing the package

@marchbold
Copy link
Contributor

You'll want to roll Firebase back not Push Notifications

apm install com.distriqt.Firebase 9.0.4

@marchbold
Copy link
Contributor

(Alternatively, we are hoping to have push notifications updated early next week so you could wait until then)

@nanhuichen
Copy link

I have the same issue and am waiting for the next update.

@raresn
Copy link
Author

raresn commented Dec 15, 2024

Yeah, can't make it work unfortunately. Will also wait :)

@raresn
Copy link
Author

raresn commented Dec 18, 2024

@marchbold will you let us know when we can try again?

@marchbold
Copy link
Contributor

You should be able to update now.. PushNotifications was updated this afternoon. Let me know if you have any issues.

@raresn
Copy link
Author

raresn commented Dec 20, 2024

worked, thx

@raresn raresn closed this as completed Dec 20, 2024
@raresn
Copy link
Author

raresn commented Dec 20, 2024

@marchbold the new size of all the extensions we have is 32 mb compared to 156 mb
Anything we did wrong or is this update this big?
new
image
old
image

@raresn raresn reopened this Dec 20, 2024
@raresn
Copy link
Author

raresn commented Dec 20, 2024

Also noticed the XML went half in size.
Just checking with you if these changes were expected.

Another thing I noticed is that the following code is totally missing from the xml now.

<receiver
      android:name="com.google.android.gms.measurement.AppMeasurementInstallReferrerReceiver"
      android:enabled="true"
      android:exported="true"
      android:permission="android.permission.INSTALL_PACKAGES">;
      <intent-filter>;
        <action android:name="com.android.vending.INSTALL_REFERRER"/>
      </intent-filter>;
    </receiver>;

Can you confirm we don't need it anymore?

@marchbold
Copy link
Contributor

Yes this is a major change in our extensions and you should expect major differences between them.

What xml are you comparing here? I believe the AppMeasurementInstallReferrerReceiver requirement was removed recently.

@raresn
Copy link
Author

raresn commented Dec 23, 2024

The xml we had in our previous release, the one that had all the "old" ANEs
Will try this one out and see how it goes

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

3 participants