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

fix(iOS): Remove unused code #247

Merged
merged 1 commit into from
Dec 19, 2018
Merged

Conversation

jcesarmobile
Copy link
Member

No description provided.

@mlynch mlynch merged commit bceb17a into ionic-team:master Dec 19, 2018
@jcesarmobile jcesarmobile deleted the remove-fix branch December 20, 2018 00:31
Ionitron added a commit that referenced this pull request Jan 3, 2019
# [3.0.0](v2.3.1...v3.0.0) (2019-01-03)

### Bug Fixes

* **iOS:** Remove unused code ([#247](#247)) ([bceb17a](bceb17a))

### Features

* Allows configuration of Mixed Content Mode ([#240](#240)) ([486d412](486d412)), closes [#231](#231)
* **Android:** Implement ionic-file and ionic-content urls ([#242](#242)) ([8ef0c30](8ef0c30)), closes [#204](#204) [#183](#183)
* **iOS:** Remove GCDWebServer ([#244](#244)) ([0dee0cf](0dee0cf))
* **WebViewLocalServer.java:** return 404 error code when a local file is not found ([#217](#217)) ([f7a551e](f7a551e)), closes [#216](#216)

### BREAKING CHANGES

* **iOS:** Sets deployment-target to 11, so will only work on iOS 11+

* Address changes
* changes the default from 1 (never) to 0 (always)
* **WebViewLocalServer.java:** Until now, the Android part of the plugin was returning a 200 http code even though
the requested file didn't exist. This behavior was inconsistent with the historical behavior of the
iOS webView. This change makes them both work in the same manner but introduces a breaking change
for the current Android users that are expecting a 200 http code no matter what and are testing the
not found error just by checking if the body is null.
@Ionitron
Copy link
Collaborator

Ionitron commented Jan 3, 2019

🎉 This PR is included in version 3.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

None yet

3 participants