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

feat: print unhandledrejection log #389

Merged
merged 5 commits into from
May 13, 2021

Conversation

zimv
Copy link
Contributor

@zimv zimv commented May 4, 2021

unhandledrejection

src/log/default.js 增加 unhandledrejection 监听事件。

当promise触发reject,并且没有注册rejection处理函数时,浏览器默认会在控制台打印错误。
当前新增特性支持vconsole控制台中,同样打印reject信息。

开发和验证

dev/log.html 增加promiseReject按钮
运行截图:
image

#254
#188

@zimv
Copy link
Contributor Author

zimv commented May 4, 2021

在小程序的运行环境是没有window的,但是vconsole内部有大量window我不知道是不是因为作为小程序内置插件,权限会多一些。为保险起见,代码中使用以下判断:

if ( !(tool.isWindow(window) && tool.isFunction(window.addEventListener)) ) {
      return
 }

@Maizify
Copy link
Collaborator

Maizify commented May 8, 2021

在小程序的运行环境是没有window的,但是vconsole内部有大量window我不知道是不是因为作为小程序内置插件,权限会多一些。为保险起见,代码中使用以下判断:

if ( !(tool.isWindow(window) && tool.isFunction(window.addEventListener)) ) {
      return
 }

感谢贡献代码~window对象问题,之前我也发现到,不过涉及到的文件比较多,我找个机会再统一改吧。

@zimv
Copy link
Contributor Author

zimv commented May 8, 2021

在小程序的运行环境是没有window的,但是vconsole内部有大量window我不知道是不是因为作为小程序内置插件,权限会多一些。为保险起见,代码中使用以下判断:

if ( !(tool.isWindow(window) && tool.isFunction(window.addEventListener)) ) {
      return
 }

感谢贡献代码~window对象问题,之前我也发现到,不过涉及到的文件比较多,我找个机会再统一改吧。

还有很多事情可以做哈,比如资源错误,性能消耗等都可以添加到日志,我这边可以持续输出,主要有些问题 想咨询一下

@Maizify
Copy link
Collaborator

Maizify commented May 12, 2021

还有很多事情可以做哈,比如资源错误,性能消耗等都可以添加到日志,我这边可以持续输出,方便的话加个微信吧:wx_zimu。主要有些问题 想咨询一下

可以先QQ群联系,回头我再搞个微信群吧。

@zimv
Copy link
Contributor Author

zimv commented May 12, 2021

还有很多事情可以做哈,比如资源错误,性能消耗等都可以添加到日志,我这边可以持续输出,方便的话加个微信吧:wx_zimu。主要有些问题 想咨询一下

可以先QQ群联系,回头我再搞个微信群吧。

qq之前我加了 没通过。。无响应,我再加一次

@Maizify Maizify merged commit 8e05ea5 into Tencent:dev May 13, 2021
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.

2 participants