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

Update Version to 4.3.0.0.0 #25

Merged
merged 3 commits into from
Jul 30, 2024
Merged

Update Version to 4.3.0.0.0 #25

merged 3 commits into from
Jul 30, 2024

Conversation

helium-service
Copy link
Contributor

@helium-service helium-service commented Jul 23, 2024

Adapter version '4.3.0.0.0', partner version '~> 3.0.0'

One change to the code: we're requesting a key from the BidMachine SDK and sending it to the backend so they can use the corresponding endpoint URL for bidding.

@helium-service helium-service requested review from a team, bwised, ricealexanderb, daniel-barros, kendallrogers and CB-HaoxinLi and removed request for a team July 23, 2024 09:03
@@ -15,7 +15,7 @@ Pod::Spec.new do |spec|

# Minimum supported versions
spec.swift_version = '5.1'
spec.ios.deployment_target = '12.0'
spec.ios.deployment_target = '13.0'

Choose a reason for hiding this comment

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

Do we need to upgrade? BidMachine 3.0.0 supports iOS 12+ and this adapter is not on 5.x yet.
https://github.com/CocoaPods/Specs/blob/master/Specs/4/0/5/BidMachine/3.0.0/BidMachine.podspec.json

Copy link
Member

Choose a reason for hiding this comment

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

+1
It's not uncommon for ad SDKs to have a minimum supported version while keeping an older one as the podspec deployment target. This way they can still be integrated in apps that have to support old OS versions, although they don't serve ads for them.

Copy link
Member

Choose a reason for hiding this comment

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

The version should be the same as the partner podspec.

Copy link
Contributor

Choose a reason for hiding this comment

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

Lies! Lies in the podspec!
Screenshot 2024-07-26 at 12 56 23 PM

Choose a reason for hiding this comment

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

That's amazing 😂

@ricealexanderb ricealexanderb merged commit 5d00db1 into main Jul 30, 2024
2 checks passed
@ricealexanderb ricealexanderb deleted the release/4.3.0.0.0 branch July 30, 2024 17:31
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