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

Migrate Stripe3DS2.framework to libStripe3DS2.a #1246

Merged
merged 14 commits into from
Jul 25, 2019
Merged

Migrate Stripe3DS2.framework to libStripe3DS2.a #1246

merged 14 commits into from
Jul 25, 2019

Conversation

davidme-stripe
Copy link
Contributor

Summary

Removes Stripe3DS2.framework in favor of a static libStripe3DS2.a.

Motivation

Nested frameworks are not allowed on the iOS App Store.

Testing

  • Tested dropping Stripe.framework into a sample app
  • Tested sample app with Carthage
  • Tested sample app with CocoaPods
  • Passed the sample app through App Store validation

Copy link
Collaborator

@yuki-stripe yuki-stripe left a comment

Choose a reason for hiding this comment

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

Everything looks good in my testing.

Want to write a quick high level blurb in CHANGELOG.md about this?
Also, do the example apps in Stripe.xcworkspace still build as well?

LIBRARY_SEARCH_PATHS = (
"$(inherited)",
"$(PROJECT_DIR)/InternalFrameworks",
);
OTHER_LDFLAGS = "-ObjC";
DEVELOPMENT_TEAM = "";
Copy link
Collaborator

Choose a reason for hiding this comment

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

Delete this line? I'm like 50% sure this makes fauxpas warn for some reason.

@davidme-stripe davidme-stripe merged commit 772737a into stripe:master Jul 25, 2019
@alvincrisuy
Copy link

I'm still encountering an issue with version 16.0.1
Advertising Identifier (IDFA) I reverted to 15.0.1 that works fine.

@davidme-stripe
Copy link
Contributor Author

Hi @alvincrisuy, thanks for the report. We’re investigating this and should have a solution for you soon.

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

Successfully merging this pull request may close these issues.

4 participants