{"pull_request":{"state":"open","deletions":469,"user":{"avatar_url":"https://secure.gravatar.com/avatar/ae959c6836366656b895069d75ad09bf?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"ae959c6836366656b895069d75ad09bf","url":"https://api.github.com/users/edavis10","login":"edavis10","id":1621},"merged_by":null,"updated_at":"2011-11-13T00:33:37Z","closed_at":null,"base":{"ref":"unstable","user":{"avatar_url":"https://secure.gravatar.com/avatar/3ac82b4bbc1a82588293c330934117e3?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-orgs.png","gravatar_id":"3ac82b4bbc1a82588293c330934117e3","url":"https://api.github.com/users/chiliproject","login":"chiliproject","id":559589},"repo":{"name":"chiliproject","pushed_at":"2011-11-08T10:42:44Z","fork":false,"master_branch":null,"open_issues":33,"updated_at":"2011-11-12T06:36:45Z","homepage":"https://www.chiliproject.org/","clone_url":"https://github.com/chiliproject/chiliproject.git","private":false,"svn_url":"https://svn.github.com/chiliproject/chiliproject","language":"Ruby","forks":89,"url":"https://api.github.com/repos/chiliproject/chiliproject","owner":{"avatar_url":"https://secure.gravatar.com/avatar/3ac82b4bbc1a82588293c330934117e3?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-orgs.png","gravatar_id":"3ac82b4bbc1a82588293c330934117e3","url":"https://api.github.com/users/chiliproject","login":"chiliproject","id":559589},"html_url":"https://github.com/chiliproject/chiliproject","description":"ChiliProject is a web based project management system built on Ruby on Rails","size":1068,"id":1320810,"git_url":"git://github.com/chiliproject/chiliproject.git","watchers":280,"created_at":"2011-02-02T16:06:12Z","ssh_url":"git@github.com:chiliproject/chiliproject.git"},"label":"chiliproject:unstable","sha":"1897459b76d2daafd957ce24e0b1654e717d30a5"},"comments":0,"diff_url":"https://github.com/chiliproject/chiliproject/pull/122.diff","commits":31,"head":{"ref":"ticket/692/new-design-for-review","user":{"avatar_url":"https://secure.gravatar.com/avatar/ae959c6836366656b895069d75ad09bf?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"ae959c6836366656b895069d75ad09bf","url":"https://api.github.com/users/edavis10","login":"edavis10","id":1621},"repo":{"name":"chiliproject","pushed_at":"2011-11-13T00:24:44Z","fork":true,"master_branch":null,"open_issues":0,"updated_at":"2011-11-13T00:24:44Z","homepage":"https://www.chiliproject.org/","clone_url":"https://github.com/edavis10/chiliproject.git","private":false,"svn_url":"https://svn.github.com/edavis10/chiliproject","language":"Ruby","forks":2,"url":"https://api.github.com/repos/edavis10/chiliproject","owner":{"avatar_url":"https://secure.gravatar.com/avatar/ae959c6836366656b895069d75ad09bf?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"ae959c6836366656b895069d75ad09bf","url":"https://api.github.com/users/edavis10","login":"edavis10","id":1621},"html_url":"https://github.com/edavis10/chiliproject","description":"Private ChiliProject fork. WARNING: feature/* ticket/* and bug/* branches are rebased arbitrarily.","size":468,"id":1320859,"git_url":"git://github.com/edavis10/chiliproject.git","watchers":17,"created_at":"2011-02-02T16:19:36Z","ssh_url":"git@github.com:edavis10/chiliproject.git"},"label":"edavis10:ticket/692/new-design-for-review","sha":"65d485dbde9b1910681e8324ee60d182b677af9b"},"additions":1923,"url":"https://api.github.com/repos/chiliproject/chiliproject/pulls/122","merged":false,"merged_at":null,"body":"Here is my results of rebasing [pull request 121](https://github.com/chiliproject/chiliproject/pull/121) to:\r\n\r\n* fix unclear commit logs and add the issue prefixes\r\n* remove merge commits from other branches\r\n* reorder commits into logically sequence (see this [gist](https://gist.github.com/1361097) for one version)\r\n* squashing related commits into a single larger commit\r\n* removing unnecessary reverts\r\n\r\nI've also done a manual diff of the layout, application.css, and application.js to pull request 121 to make sure nothing major was missed.\r\n\r\nSince there is so much here, lets try to keep the code review in the [issue](https://www.chiliproject.org/issues/692). We should review this like so:\r\n\r\n1. should each commit be included or not?\r\n2. for code changes as a whole, is each line acceptable (security, code standards, etc)\r\n3. does the changes make sense? Like is this a good change or should it be different.\r\n\r\n(I'm going to copy this into the main issue too so we can keep the discussion there).\r\n\r\nEric Davis","review_comments":0,"_links":{"comments":{"href":"https://api.github.com/repos/chiliproject/chiliproject/issues/122/comments"},"html":{"href":"https://github.com/chiliproject/chiliproject/pull/122"},"review_comments":{"href":"https://api.github.com/repos/chiliproject/chiliproject/pulls/122/comments"},"self":{"href":"https://api.github.com/repos/chiliproject/chiliproject/pulls/122"}},"html_url":"https://github.com/chiliproject/chiliproject/pull/122","issue_url":"https://github.com/chiliproject/chiliproject/issues/122","title":"New design for review #692","number":122,"mergeable":null,"id":489333,"changed_files":54,"patch_url":"https://github.com/chiliproject/chiliproject/pull/122.patch","created_at":"2011-11-13T00:33:37Z"},"action":"opened","number":122}