Skip to content

Conversation

StevenMasini
Copy link

New Pull Request Checklist

Issue Description

Adding subspec Parse/FacebookUtils to a project will generate linking compilation issues, caused by the missing dependency FBSDKCoreKit.

Related issue: #1665

Approach

TODOs before merging

  • Add tests
  • Add entry to changelog
  • Add changes to documentation (guides, repository pages, in-code descriptions)
@parse-github-assistant
Copy link

parse-github-assistant bot commented Oct 4, 2022

Thanks for opening this pull request!

  • 🎉 We are excited about your hands-on contribution!
@mtrezza mtrezza linked an issue Oct 4, 2022 that may be closed by this pull request
4 tasks
@mtrezza mtrezza changed the title Add FBSDKCoreKit dependency to Parse/FacebookUtils fix: implementation via CocoaPods fails due to missing FBSDKCoreKit dependency in Parse/FacebookUtils Oct 4, 2022
@danipralea
Copy link

is there any way we can merge this? @mtrezza

@mtrezza
Copy link
Member

mtrezza commented Oct 26, 2022

@StevenMasini Hasn't requested a review yet, so I'm not sure, is this ready? Does it fix the issue for you?

@danipralea
Copy link

I don't know to be honest. I can't say for sure until it gets merged. or maybe manually indicate his branch in Podfile?

@StevenMasini
Copy link
Author

@mtrezza I am just wondering what's wrong with the ci/macos? I don't think my change caused that issue. Otherwise, I think it's ready to merge.

@mtrezza
Copy link
Member

mtrezza commented Oct 26, 2022

Yes, doesn't seem related, I'll go ahead and merge.

@mtrezza mtrezza changed the title fix: implementation via CocoaPods fails due to missing FBSDKCoreKit dependency in Parse/FacebookUtils fix: implementation via CocoaPods fails due to missing FBSDKCoreKit dependency Oct 26, 2022
Copy link
Member

@mtrezza mtrezza left a comment

Choose a reason for hiding this comment

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

Looks good!

@mtrezza mtrezza merged commit ac8a4fa into parse-community:master Oct 26, 2022
parseplatformorg pushed a commit that referenced this pull request Oct 26, 2022
## [1.19.4](1.19.3...1.19.4) (2022-10-26) ### Bug Fixes * implementation via CocoaPods fails due to missing `FBSDKCoreKit` dependency ([#1666](#1666)) ([ac8a4fa](ac8a4fa))
@parseplatformorg
Copy link
Contributor

🎉 This change has been released in version 1.19.4

@parseplatformorg parseplatformorg added the state:released Released as stable version label Oct 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

state:released Released as stable version

4 participants