Closed shaoshing closed 8 years ago
Looking good! Back to @shaoshing for a little more earth-moving.
Hi @iwehrman I've updated the PR to address your comments. The highlight are as follows:
descriptor._getAsync
and descriptor._batchPlayAsync
Back to you to review again. Thank you.
Hi @iwehrman, I addressed the transfer naming issue and exclude the debug
module in production mode. Back to you.
the new change are tested/passed in dev and production mode.
Fantastic, thanks!
This PR makes the following change to the debugging menu items:
Moves all debugging menu items to a new top-level menu
Debug
Addes new debugging items:
1
Load Debugging Helpers
2
Log Descriptor Events
: Log descriptor events. includes events sent by PS and DS. This option is togglable.3
Log Highlights Events (togglable)
: Log Highlights tracking events. This option is togglable.