Skip to content

Specified Bolts "Tasks" in subspecs dependencies for facebook utils. … #1452

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

Merged
merged 1 commit into from
Sep 26, 2019

Conversation

tealshift
Copy link
Contributor

…This removes Bolts/AppLinks and thus UIWebView.

…This removes Bolts/AppLinks and thus UIWebView.
Copy link
Member

@noobs2ninjas noobs2ninjas 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 to me!

@tealshift
Copy link
Contributor Author

I forgot to note that this addresses issue: #1443

@drdaz
Copy link
Member

drdaz commented Sep 26, 2019

Yeah I like it. Let's merge it.

@drdaz
Copy link
Member

drdaz commented Sep 26, 2019

This actually brings it in line with the rest of the podspec. 👍🏼

@mmimeault
Copy link

Hello, could it be possible to make a new release with that change. We are trying to remove all usages of UIWebView in our project. Thanks in advance.

@benpackard
Copy link

Just to note that apps using UIWebView will not be accepted from April. https://developer.apple.com/news/?id=12232019b

@chriscborg
Copy link

Has a release been created for this fix? Or do you think it's safe to use master directly as it is. I'm still getting references to UIWebview in version 1.17.3 and unfortunately no new apps are being approved because of this.

@TomWFox
Copy link
Contributor

TomWFox commented Apr 12, 2020

No release yet, we’re working on it.

I think it’s safe to use master at the moment but @drdaz @noobs2ninjas are in a better position to answer that.

@drdaz
Copy link
Member

drdaz commented Apr 12, 2020

Yeah master should be fine. Pls do let us know if it isn't :)

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.

8 participants