{"action":"created","issue":{"url":"https://api.github.com/repos/yagihashib/TC359_OneModelRailsApp/issues/1","labels_url":"https://api.github.com/repos/yagihashib/TC359_OneModelRailsApp/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/yagihashib/TC359_OneModelRailsApp/issues/1/comments","events_url":"https://api.github.com/repos/yagihashib/TC359_OneModelRailsApp/issues/1/events","html_url":"https://github.com/yagihashib/TC359_OneModelRailsApp/issues/1","id":65477595,"number":1,"title":"Review notes","user":{"login":"chrisvfritz","id":2327556,"avatar_url":"https://avatars.githubusercontent.com/u/2327556?v=3","gravatar_id":"","url":"https://api.github.com/users/chrisvfritz","html_url":"https://github.com/chrisvfritz","followers_url":"https://api.github.com/users/chrisvfritz/followers","following_url":"https://api.github.com/users/chrisvfritz/following{/other_user}","gists_url":"https://api.github.com/users/chrisvfritz/gists{/gist_id}","starred_url":"https://api.github.com/users/chrisvfritz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chrisvfritz/subscriptions","organizations_url":"https://api.github.com/users/chrisvfritz/orgs","repos_url":"https://api.github.com/users/chrisvfritz/repos","events_url":"https://api.github.com/users/chrisvfritz/events{/privacy}","received_events_url":"https://api.github.com/users/chrisvfritz/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-03-31T15:25:36Z","updated_at":"2015-04-01T04:35:25Z","closed_at":null,"body":"I see a few issues I'd like you to fix here.\r\n\r\n### Form errors styling\r\n\r\nThe styling on the navigation menu seems to also be affecting the form errors. Can you tweak your styling to make sure it's not overreaching?\r\n\r\n### Avgfanduelpoints\r\n\r\nI think `Avgfanduelpoints` could be renamed to something a little more readable to visitors. I was also able to enter in a negative number, which I don't *think* is desirable. [This Stack Overflow question](http://stackoverflow.com/questions/4416278/how-to-implement-min-max-validator-in-rails-3) should help you figure out how to fix this."},"comment":{"url":"https://api.github.com/repos/yagihashib/TC359_OneModelRailsApp/issues/comments/88347081","html_url":"https://github.com/yagihashib/TC359_OneModelRailsApp/issues/1#issuecomment-88347081","issue_url":"https://api.github.com/repos/yagihashib/TC359_OneModelRailsApp/issues/1","id":88347081,"user":{"login":"chrisvfritz","id":2327556,"avatar_url":"https://avatars.githubusercontent.com/u/2327556?v=3","gravatar_id":"","url":"https://api.github.com/users/chrisvfritz","html_url":"https://github.com/chrisvfritz","followers_url":"https://api.github.com/users/chrisvfritz/followers","following_url":"https://api.github.com/users/chrisvfritz/following{/other_user}","gists_url":"https://api.github.com/users/chrisvfritz/gists{/gist_id}","starred_url":"https://api.github.com/users/chrisvfritz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chrisvfritz/subscriptions","organizations_url":"https://api.github.com/users/chrisvfritz/orgs","repos_url":"https://api.github.com/users/chrisvfritz/repos","events_url":"https://api.github.com/users/chrisvfritz/events{/privacy}","received_events_url":"https://api.github.com/users/chrisvfritz/received_events","type":"User","site_admin":false},"created_at":"2015-04-01T04:35:25Z","updated_at":"2015-04-01T04:35:25Z","body":"It looks like the form errors are still appearing inline, instead of in a normal list. To give you more context, the form errors would normally display in [a list like this](https://www.dropbox.com/s/firbgrr9nauwhq0/Screenshot%202015-04-01%2000.28.19.png?dl=0), but some overreaching styles in your `application.css` are making them appear [more like this](https://github.com/yagihashib/TC359_OneModelRailsApp/blob/master/app/assets/stylesheets/application.css).\r\n\r\nThen the problem with readable labels for `Avgfanduelpoints` isn't really fixed by renaming the `avgfanduelpoints` attribute to `avgpoints`. I'd expect to see something more like `Average Points` when it's displayed to the visitor in a form label or table header, for example. The same goes for `Nbaplayer` actually. I can see that you're showing `NBA Player` sometimes, but not everywhere."}}