{"action":"created","issue":{"url":"https://api.github.com/repos/code-dot-org/code-dot-org/issues/8071","repository_url":"https://api.github.com/repos/code-dot-org/code-dot-org","labels_url":"https://api.github.com/repos/code-dot-org/code-dot-org/issues/8071/labels{/name}","comments_url":"https://api.github.com/repos/code-dot-org/code-dot-org/issues/8071/comments","events_url":"https://api.github.com/repos/code-dot-org/code-dot-org/issues/8071/events","html_url":"https://github.com/code-dot-org/code-dot-org/pull/8071","id":151269700,"number":8071,"title":"Applab export build fix","user":{"login":"pcardune","id":124827,"avatar_url":"https://avatars.githubusercontent.com/u/124827?v=3","gravatar_id":"","url":"https://api.github.com/users/pcardune","html_url":"https://github.com/pcardune","followers_url":"https://api.github.com/users/pcardune/followers","following_url":"https://api.github.com/users/pcardune/following{/other_user}","gists_url":"https://api.github.com/users/pcardune/gists{/gist_id}","starred_url":"https://api.github.com/users/pcardune/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pcardune/subscriptions","organizations_url":"https://api.github.com/users/pcardune/orgs","repos_url":"https://api.github.com/users/pcardune/repos","events_url":"https://api.github.com/users/pcardune/events{/privacy}","received_events_url":"https://api.github.com/users/pcardune/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"Bjvanminnen","id":1767466,"avatar_url":"https://avatars.githubusercontent.com/u/1767466?v=3","gravatar_id":"","url":"https://api.github.com/users/Bjvanminnen","html_url":"https://github.com/Bjvanminnen","followers_url":"https://api.github.com/users/Bjvanminnen/followers","following_url":"https://api.github.com/users/Bjvanminnen/following{/other_user}","gists_url":"https://api.github.com/users/Bjvanminnen/gists{/gist_id}","starred_url":"https://api.github.com/users/Bjvanminnen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Bjvanminnen/subscriptions","organizations_url":"https://api.github.com/users/Bjvanminnen/orgs","repos_url":"https://api.github.com/users/Bjvanminnen/repos","events_url":"https://api.github.com/users/Bjvanminnen/events{/privacy}","received_events_url":"https://api.github.com/users/Bjvanminnen/received_events","type":"User","site_admin":false},"milestone":null,"comments":5,"created_at":"2016-04-27T00:23:48Z","updated_at":"2016-04-27T15:54:06Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/code-dot-org/code-dot-org/pulls/8071","html_url":"https://github.com/code-dot-org/code-dot-org/pull/8071","diff_url":"https://github.com/code-dot-org/code-dot-org/pull/8071.diff","patch_url":"https://github.com/code-dot-org/code-dot-org/pull/8071.patch"},"body":"In order to fix this, I switched to using the prebuilt version of jszip which comes with the jszip npm package. This is a workaround and not the ideal fix. I'm not sure what the ideal fix is though. It seems like many of our js dependencies (including jszip) depend on the `isarray` package and for some reason browserify doesn't include the right `isarray` package in the resulting bundle (or at least it doesn't include it at the right path) so you get an error when the page loads. By using the prebuilt version of jszip, we get around this problem.\r\n\r\nMy suspicion is the webpack might not have this problem, so perhaps if and when we switch to webpack we can get rid of this hack."},"comment":{"url":"https://api.github.com/repos/code-dot-org/code-dot-org/issues/comments/215128801","html_url":"https://github.com/code-dot-org/code-dot-org/pull/8071#issuecomment-215128801","issue_url":"https://api.github.com/repos/code-dot-org/code-dot-org/issues/8071","id":215128801,"user":{"login":"Bjvanminnen","id":1767466,"avatar_url":"https://avatars.githubusercontent.com/u/1767466?v=3","gravatar_id":"","url":"https://api.github.com/users/Bjvanminnen","html_url":"https://github.com/Bjvanminnen","followers_url":"https://api.github.com/users/Bjvanminnen/followers","following_url":"https://api.github.com/users/Bjvanminnen/following{/other_user}","gists_url":"https://api.github.com/users/Bjvanminnen/gists{/gist_id}","starred_url":"https://api.github.com/users/Bjvanminnen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Bjvanminnen/subscriptions","organizations_url":"https://api.github.com/users/Bjvanminnen/orgs","repos_url":"https://api.github.com/users/Bjvanminnen/repos","events_url":"https://api.github.com/users/Bjvanminnen/events{/privacy}","received_events_url":"https://api.github.com/users/Bjvanminnen/received_events","type":"User","site_admin":false},"created_at":"2016-04-27T15:54:06Z","updated_at":"2016-04-27T15:54:06Z","body":"Change seems ok to me if it works. Might be worth adding a comment above the require explaining why we're requiring this version."}}