{"action":"created","issue":{"url":"https://api.github.com/repos/rmurphey/js-assessment/issues/122","labels_url":"https://api.github.com/repos/rmurphey/js-assessment/issues/122/labels{/name}","comments_url":"https://api.github.com/repos/rmurphey/js-assessment/issues/122/comments","events_url":"https://api.github.com/repos/rmurphey/js-assessment/issues/122/events","html_url":"https://github.com/rmurphey/js-assessment/pull/122","id":75740967,"number":122,"title":"WIP: Updates to #112 refactor","user":{"login":"jmeas","id":2322305,"avatar_url":"https://avatars.githubusercontent.com/u/2322305?v=3","gravatar_id":"","url":"https://api.github.com/users/jmeas","html_url":"https://github.com/jmeas","followers_url":"https://api.github.com/users/jmeas/followers","following_url":"https://api.github.com/users/jmeas/following{/other_user}","gists_url":"https://api.github.com/users/jmeas/gists{/gist_id}","starred_url":"https://api.github.com/users/jmeas/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jmeas/subscriptions","organizations_url":"https://api.github.com/users/jmeas/orgs","repos_url":"https://api.github.com/users/jmeas/repos","events_url":"https://api.github.com/users/jmeas/events{/privacy}","received_events_url":"https://api.github.com/users/jmeas/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-05-12T21:14:36Z","updated_at":"2015-05-12T21:49:43Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/rmurphey/js-assessment/pulls/122","html_url":"https://github.com/rmurphey/js-assessment/pull/122","diff_url":"https://github.com/rmurphey/js-assessment/pull/122.diff","patch_url":"https://github.com/rmurphey/js-assessment/pull/122.patch"},"body":"Don't merge this yet! I'll be adding updates as I test #112.\r\n\r\n#### Updates\r\n\r\n- Adding jQuery to the package.json. I received an error when I tried to load the tests in the browser 'cuz it couldn't find jquery.\r\n\r\n#### To investigate\r\n\r\nThe following solutions are failing:\r\n\r\n- logical operators\r\n  - you should be able to work with logical and (**✘ This works on master**)\r\n- ~~permutation~~\r\n  - ~~you should be able to return the nth number in a fibonacci sequence~~\r\n  - ~~you should be able to return the set of all valid combinations of n pairs of parentheses.~~\r\n\r\nI'm investigating whether this is the case on master, as well, or if this was a regression introduced in this PR."},"comment":{"url":"https://api.github.com/repos/rmurphey/js-assessment/issues/comments/101431614","html_url":"https://github.com/rmurphey/js-assessment/pull/122#issuecomment-101431614","issue_url":"https://api.github.com/repos/rmurphey/js-assessment/issues/122","id":101431614,"user":{"login":"jmeas","id":2322305,"avatar_url":"https://avatars.githubusercontent.com/u/2322305?v=3","gravatar_id":"","url":"https://api.github.com/users/jmeas","html_url":"https://github.com/jmeas","followers_url":"https://api.github.com/users/jmeas/followers","following_url":"https://api.github.com/users/jmeas/following{/other_user}","gists_url":"https://api.github.com/users/jmeas/gists{/gist_id}","starred_url":"https://api.github.com/users/jmeas/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jmeas/subscriptions","organizations_url":"https://api.github.com/users/jmeas/orgs","repos_url":"https://api.github.com/users/jmeas/repos","events_url":"https://api.github.com/users/jmeas/events{/privacy}","received_events_url":"https://api.github.com/users/jmeas/received_events","type":"User","site_admin":false},"created_at":"2015-05-12T21:49:43Z","updated_at":"2015-05-12T21:49:43Z","body":"There were 3 failing tests. Two are [missing answers](https://github.com/rmurphey/js-assessment-answers/issues/33); the other does work on master. Working to fix it now!"}}