Skip to content
This repository has been archived by the owner on Jun 3, 2021. It is now read-only.

[WEEX-131][ios]URL property in error event of web component may not be the real URL, fix it. #896

Closed
wants to merge 1 commit into from

Conversation

CYJB
Copy link
Contributor

@CYJB CYJB commented Nov 21, 2017

[WEEX-131][ios]URL property in error event of web component may not be the real URL, fix it.

In error callback, webview.request may return the previous loaded web page URL, use error.userInfo to get the real error URL.

And http://www.error.com exists now, change it to another URL.

…e the real URL, fix it.

In error callback, webview.request may return the previous loaded web page URL, use error.userInfo to get the real error URL.

And http://www.error.com exists now, change it to another URL.
@weex-bot
Copy link

weex-bot commented Nov 21, 2017

Messages
📖 has no android file changed.
📖 android test finished.

Generated by 🚫 dangerJS

@asfgit asfgit closed this in 805f8a8 Nov 22, 2017
@CYJB CYJB deleted the WEEX-131 branch November 22, 2017 03:44
atomtong added a commit to atomtong/incubator-weex that referenced this pull request Nov 29, 2017
* master:
  * [doc] publish 0.17.0 release note
  [WEEX-112][iOS] make weex playground app adapt to iPhone X
  close apache#896
  * [test] bug fix , compatibility of maraca can't find element
  [WEEX-131][ios]URL property in error event of web component may not be the real URL, fix it.
  close apache#822
  Add mavenLocal before jcenter repository
  Remove maven central repository, use only jcenter
  close apache#790,close apache#796,close apache#801,close apache#803
  close apache#884,close apache#886,close apache#889
  -[test] close apache#887
  resolve apache#887
  [WEEX-132][iOS] support text writing direction from right to left
  [WEEX-133][iOS] support word-wrap on iOS when drawing text
  [WEEX-129][android] Appear Event Be Fired Multi Times And Event not watch be fired
  [WEEX-124][android]fix android bug
atomtong added a commit to atomtong/incubator-weex that referenced this pull request Nov 29, 2017
…eption

* release-0.16-gitlab:
  * [doc] publish 0.17.0 release note
  [WEEX-112][iOS] make weex playground app adapt to iPhone X
  close apache#896
  * [test] bug fix , compatibility of maraca can't find element
  [WEEX-131][ios]URL property in error event of web component may not be the real URL, fix it.
  close apache#822
  Add mavenLocal before jcenter repository
  Remove maven central repository, use only jcenter
  * [android] avoid crash on some device caused by RippleDrawable
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants