@justin808 I didn't see any current tests to add this to. We're currently running 5.2. I have to see about upgrading to 6.x

@martyphee Anyway I can start using this code for my app? I haven't found any other work around. Would you know how to write to the beginning of the webpack bundled file through settings in webpack config?

Coverage remained the same at 82.305% when pulling 2318c9a on martyphee:add-clearTimeout-polyfill into 3fe05f9 on shakacode:master.

I have a work around for anyone interested.

cd client
rm -rf node_modules
npm install
npm shrinkwrap

Edit shrinkwrap.json. Update process like this.

"process": {
    "version": "0.11.3",
    "from": "process@>=0.11.0 <0.12.0",
    "resolved": "http://registry.npmjs.org/process/-/process-0.11.3.tgz"
}

@justin808 I have a PR out there to polyFill and I have a PR for our codebase to bump it to 6.x of react_on_rails.

I'm reviewing. Tests would be a bonus.

Review status: 0 of 1 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Review status: 0 of 1 files reviewed at latest revision, 1 unresolved discussion.

lib/react_on_rails/server_rendering_pool/exec.rb, line 137 [r1] (raw file):

function clearTimeout() {
  #{undefined_for_exec_js_logging('setTimeout')}

should be 'clearTimeout'

Comments from Reviewable

One comment.

Review status: 0 of 1 files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Just pushed update. Sorry about that.

On Sat, Jun 11, 2016 at 4:11 PM, Justin Gordon [email protected]
wrote:

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved

discussion.

Comments from Reviewable
https://reviewable.io:443/reviews/shakacode/react_on_rails/451

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#451 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AAuEtqfB2_VjEKXP7jHgeSWHM29RCPx8ks5qKyRwgaJpZM4IzlLB
.

Coverage remained the same at 82.305% when pulling 5c97165 on martyphee:add-clearTimeout-polyfill into 3fe05f9 on shakacode:master.

@martyphee Please update the Changelog.md per the examples, and I'll merge and release 6.0.4.

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Done

Coverage remained the same at 82.305% when pulling 173176e on martyphee:add-clearTimeout-polyfill into 3fe05f9 on shakacode:master.

Please see the comment and squash to one commit and I'll release. Thanks!

Reviewed 1 of 1 files at r3.
Review status: all files reviewed at latest revision, 2 unresolved discussions.

CHANGELOG.md, line 8 [r3] (raw file):

## [Unreleased]

## [6.0.4]

See the bottom of the file for where this goes.

Comments from Reviewable

Coverage remained the same at 82.305% when pulling ec2e83c on martyphee:add-clearTimeout-polyfill into 3fe05f9 on shakacode:master.

Thanks @martyphee

Review status: 1 of 2 files reviewed at latest revision, 2 unresolved discussions.

Comments from Reviewable

Reviewed 1 of 1 files at r4.
Review status: all files reviewed at latest revision, 2 unresolved discussions.

Comments from Reviewable