Skip to content
This repository was archived by the owner on May 1, 2024. It is now read-only.

[Update] Firebase for iOS v8.0.0 #485

Closed
wants to merge 57 commits into from

Conversation

dmariogatto
Copy link
Contributor

Includes

Firebase

  • All projects updated 8.0.0
  • MLKit for Firebase packages deprecated/removed (no longer supported)
    • Firebase.MLKit
    • Firebase.MLKit.Common
    • Firebase.MLKit.ModelInterpreter
    • Firebase.MLKit.NaturalLanguage
    • Firebase.MLKit.Vision
    • Note: probably need a new project/repo for the newer Google ML Kit bindings
  • Added beta binding for Firebase App Distribution (Firebase.AppDistribution) & App Check (Firebase.AppCheck)
    • excluded for now (no sample or testing done)
  • Minimum iOS version increased to 10.0

Google

  • AdMob updated to 8.5.0
  • Google Cast updated to 4.6.1
  • Google Maps & Places updated to 5.0.0
  • Google Tag Manager updated to 7.3.1
  • Deprecated/removed packages (due to outdated Firebase dependencies),
    • Google.AppIndexing
    • Google.Core
    • Google.InstanceID
    • Firebase.InstanceID
  • Minimum iOS version increased to 10.0 (due to Firebase dependencies)

Notes

Removed the Wrap methods/properties that where marked as Obsolete (seems like a major version bump is the time to do it 😅).

Added a new build property that can be used to link Firebase with Ad Id support.

<PropertyGroup>
  <FirebaseWithoutAdIdSupport>True</FirebaseWithoutAdIdSupport>
</PropertyGroup>

There were quite a few binding changes, so would definitely appreciate someone to look them over. All the samples do run & link as far as I can tell.

UMP has been separated out from AdMob (Google.UserMessagingPlatform).

Any issues let me know!

dmariogatto and others added 30 commits April 20, 2021 17:23
@mayhammf
Copy link

@dmariogatto thank you so much for working on this.
I just wanna mention that we've used your branch 711 to produce locally built firebase package (with v7.11) and were able to release a build (with FirebaseWithoutAdIdSupport=true) and Apple has approved all of our apps without any issues concerning the tracking transparency stuff.

@mayhammf
Copy link

We did have to cleanup some of the project refs in the .sln file to get it to build in Visual Studio.
This PR here includes the projects in question
dmariogatto@3c828b7

@dmariogatto
Copy link
Contributor Author

@mayhammf Glad it's helped!

I didn't notice that PR, I'll review and merge it into this branch.

@dmariogatto
Copy link
Contributor Author

@mayhammf InstanceID was removed as a dependency from Messaging in 8.0.0. You can see the difference in the "dependencies" section of the podspec files - 7.11.0 & 8.0.0.

@mayhammf
Copy link

@mayhammf InstanceID was removed as a dependency from Messaging in 8.0.0. You can see the difference in the "dependencies" section of the podspec files - 7.11.0 & 8.0.0.

Thank you @dmariogatto for taking the time to reply. I noticed that i was looking at the wrong branch after i posted my question. That's why i ended up deleting it. But it was too later already 😇

@dtaylorus
Copy link

@dmariogatto This repo appears to be abandoned. Any chance you would publish the nugets from your branch?

@dmariogatto
Copy link
Contributor Author

@dtaylorus I'd rather leave that as a last resort, it would fragment Firebase packages/documentation/samples. I don't really want to be the sole maintainer either, I can't commit that I'll always have the time/knowledge to keep this repo up-to-date.

I'll continue updating the bindings while I can & creating PRs. Hopefully at some point they will get reviewed, or there will be some indication if this repo is truly abandoned (or possibly just side-lined while teams are busy with MAUI & .NET 6).

@dtaylorus
Copy link

@dmariogatto Fair enough. In the meantime your branch is super helpful. Thanks! I submitted a pull request to your branch for some fixes required to use rewarded and adaptive banner ads.

Fixes for RewardedAd Api definitions
@dmariogatto
Copy link
Contributor Author

Now obsolete, see #489

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants