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

use StateMonitor to record&check wechaty/puppet/bridge/browser state change #46

Closed
5 tasks done
huan opened this issue Oct 15, 2016 · 1 comment
Closed
5 tasks done
Assignees
Milestone

Comments

@huan
Copy link
Member

huan commented Oct 15, 2016

now, the all above modules have the same targetState() & currentState() code, should be modulized & well tested, to make sure all the state are well monitor & managed.

  • Browser Class
  • [ ] Bridge Class
  • [ ] Server Class
  • Puppet Class
  • Wechaty Class
  • Io Class
  • IoClient Class

StateMonitor Class: https://github.com/wechaty/wechaty/blob/master/src/state-monitor.ts

after this, Watchdog should work better, and we can dig deeper for how to deal with browser problems, i.e.: killed by operation system or hang with no reason.

@huan huan added this to the v0.5 milestone Oct 15, 2016
@huan huan self-assigned this Oct 15, 2016
huan added a commit that referenced this issue Oct 25, 2016
huan added a commit that referenced this issue Oct 25, 2016
huan added a commit that referenced this issue Oct 25, 2016
huan added a commit that referenced this issue Oct 25, 2016
huan added a commit that referenced this issue Oct 25, 2016
@huan
Copy link
Member Author

huan commented Oct 25, 2016

all done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant