{"action":"created","issue":{"url":"https://api.github.com/repos/reactjs/redux/issues/1779","repository_url":"https://api.github.com/repos/reactjs/redux","labels_url":"https://api.github.com/repos/reactjs/redux/issues/1779/labels{/name}","comments_url":"https://api.github.com/repos/reactjs/redux/issues/1779/comments","events_url":"https://api.github.com/repos/reactjs/redux/issues/1779/events","html_url":"https://github.com/reactjs/redux/issues/1779","id":157369069,"number":1779,"title":"Redux docs are telling people to use ternary operators in inline styles.","user":{"login":"openSourceBugs","id":5646010,"avatar_url":"https://avatars.githubusercontent.com/u/5646010?v=3","gravatar_id":"","url":"https://api.github.com/users/openSourceBugs","html_url":"https://github.com/openSourceBugs","followers_url":"https://api.github.com/users/openSourceBugs/followers","following_url":"https://api.github.com/users/openSourceBugs/following{/other_user}","gists_url":"https://api.github.com/users/openSourceBugs/gists{/gist_id}","starred_url":"https://api.github.com/users/openSourceBugs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/openSourceBugs/subscriptions","organizations_url":"https://api.github.com/users/openSourceBugs/orgs","repos_url":"https://api.github.com/users/openSourceBugs/repos","events_url":"https://api.github.com/users/openSourceBugs/events{/privacy}","received_events_url":"https://api.github.com/users/openSourceBugs/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2016-05-29T04:45:29Z","updated_at":"2016-05-29T05:30:58Z","closed_at":"2016-05-29T05:25:19Z","body":"This seems utterly unuseful (to say the least):\r\n\r\n   style={{\r\n      textDecoration: completed ? 'line-through' : 'none'\r\n    }}\r\n\r\nthis is found on\r\nhttp://redux.js.org/docs/basics/ExampleTodoList.html\r\n\r\nproblems with this:\r\n1.  inline css\r\n2.  inline css\r\n3.  inline css\r\n4.  INLINE CSS WITH A TERNARY OPERATOR IN IT.  This can only be tested from an end user's point of view.  I must physically open a browser, click around and see if something happens.  Logic with persistent state inside presentation can really only be tested heavily via end to end testing since it involves view, model, and controller all in one. But this is only the front end.\r\n5.  Shouldn't \"completed\" be persisted?  This seems like a violation of separation of concerns.  Isn't this supposed to be the view layer? Not the Postgres layer?"},"comment":{"url":"https://api.github.com/repos/reactjs/redux/issues/comments/222343725","html_url":"https://github.com/reactjs/redux/issues/1779#issuecomment-222343725","issue_url":"https://api.github.com/repos/reactjs/redux/issues/1779","id":222343725,"user":{"login":"openSourceBugs","id":5646010,"avatar_url":"https://avatars.githubusercontent.com/u/5646010?v=3","gravatar_id":"","url":"https://api.github.com/users/openSourceBugs","html_url":"https://github.com/openSourceBugs","followers_url":"https://api.github.com/users/openSourceBugs/followers","following_url":"https://api.github.com/users/openSourceBugs/following{/other_user}","gists_url":"https://api.github.com/users/openSourceBugs/gists{/gist_id}","starred_url":"https://api.github.com/users/openSourceBugs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/openSourceBugs/subscriptions","organizations_url":"https://api.github.com/users/openSourceBugs/orgs","repos_url":"https://api.github.com/users/openSourceBugs/repos","events_url":"https://api.github.com/users/openSourceBugs/events{/privacy}","received_events_url":"https://api.github.com/users/openSourceBugs/received_events","type":"User","site_admin":false},"created_at":"2016-05-29T05:30:58Z","updated_at":"2016-05-29T05:30:58Z","body":"May I quote this in the future?\r\n\"...inline styles are a perfectly valid way to do CSS\""}}
{"actor":{"display_login":"openSourceBugs"}}