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

[android] js exception adapter change to list. so that can handler m… #395

Closed
wants to merge 1 commit into from
Closed

[android] js exception adapter change to list. so that can handler m… #395

wants to merge 1 commit into from

Conversation

wispy316
Copy link
Contributor

[android] js exception adapter change to list. so that can handler more than one listener.

Now faced a problem is
WeexErrorCallback (used by Motu weex js exception caught)
is conflict with
JSExceptionGlobalEventReport (added in taobao_weex_adapter to caught js exception and throw it to js, we use JSTracker to log exceptions.)

…re than one listener. Now faced a problem is WeexErrorCallback (used by Motu weex js exception caught) is conflict with JSExceptionGlobalEventReport (added in taobao_weex_adapter to caught js exception and throw it to js, we use JSTracker to log exceptions.)
@weex-bot
Copy link

Fails
🚫 Running your Dangerfile has Failed

Danger has errored

Error: TypeError

TypeError: diff.match is not a function
    at Object.<anonymous> (dangerfile.js:109:24)
    at Runtime._execModule (/home/travis/build/apache/incubator-weex/node_modules/jest-runtime/build/index.js:447:13)
    at Runtime.requireModule (/home/travis/build/apache/incubator-weex/node_modules/jest-runtime/build/index.js:295:14)
    at /home/travis/build/apache/incubator-weex/node_modules/danger/distribution/runner/DangerfileRunner.js:136:33
    at ensureCleanDangerfile (/home/travis/build/apache/incubator-weex/node_modules/danger/distribution/runner/DangerfileRunner.js:176:5)
    at Object.<anonymous> (/home/travis/build/apache/incubator-weex/node_modules/danger/distribution/runner/DangerfileRunner.js:135:21)
    at step (/home/travis/build/apache/incubator-weex/node_modules/danger/distribution/runner/DangerfileRunner.js:32:23)
    at Object.next (/home/travis/build/apache/incubator-weex/node_modules/danger/distribution/runner/DangerfileRunner.js:13:53)
    at /home/travis/build/apache/incubator-weex/node_modules/danger/distribution/runner/DangerfileRunner.js:7:71
    at __awaiter (/home/travis/build/apache/incubator-weex/node_modules/danger/distribution/runner/DangerfileRunner.js:3:12)

Generated by 🚫 dangerJS

@wispy316
Copy link
Contributor Author

wispy316 commented Jun 1, 2017

do it in WeexErrorCallBack at crashreportadapterLoader

@wispy316 wispy316 closed this Jun 1, 2017
@wispy316 wispy316 deleted the feature-android-js-error-list branch June 15, 2017 02:40
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.

None yet

2 participants