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.

Sign up for GitHub

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

Jump to bottom

在红米机子上会报错 #40

Closed
whxaxes opened this issue Jun 17, 2016 · 5 comments
Closed

在红米机子上会报错 #40

whxaxes opened this issue Jun 17, 2016 · 5 comments

Comments

@whxaxes
Copy link

whxaxes commented Jun 17, 2016

rt

image
@Maizify
Copy link
Collaborator

Maizify commented Jun 18, 2016

请问用的是哪个vConsole版本？

@whxaxes
Copy link
Author

whxaxes commented Jun 20, 2016

2.0.1，直接在你们仓库里的dist目录下copy过来的

Maizify added a commit to Maizify/vConsole that referenced this issue Jun 20, 2016
@Maizify
fix issue Tencent#40, empty stack in window.onerror
5f866d4
Maizify added a commit that referenced this issue Jun 20, 2016
@Maizify
Merge pull request #41 from Maizify/dev
6e1a603 
Fix issue #40
@Maizify
Copy link
Collaborator

Maizify commented Jun 20, 2016

修了下，更新到了 dev 分支中，版本号变成 v2.0.2-pre，麻烦下载 dev 分支下的 dist/vconsole.min.js 再试试看？

@whxaxes
Copy link
Author

whxaxes commented Jun 20, 2016

非常感谢！！

image

尝试了一下，用是可以用了，不过还是会抛出异常，可能跟我用了browser-sync的原因有关

我看了一下，发现错误出在了这一行，排查是发现获取xhr的status属性的时候抛的错
image

然后在stackoverflow上看到
image

然后我改成这样就正常了
image

仅供参考

@Maizify
Copy link
Collaborator

Maizify commented Jun 20, 2016

好的，感谢反馈！

Maizify added a commit to Maizify/vConsole that referenced this issue Jun 20, 2016
@Maizify
fix issue Tencent#40, update xhr.status only if readyState==4
0367866
Maizify added a commit that referenced this issue Jun 20, 2016
@Maizify
Merge pull request #42 from Maizify/dev
69dba6d 
fix issue #40, update xhr.status only if readyState==4
@Maizify Maizify closed this as completed Jun 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Maizify @whxaxes