{"action":"created","issue":{"url":"https://api.github.com/repos/balloob/home-assistant/issues/1529","repository_url":"https://api.github.com/repos/balloob/home-assistant","labels_url":"https://api.github.com/repos/balloob/home-assistant/issues/1529/labels{/name}","comments_url":"https://api.github.com/repos/balloob/home-assistant/issues/1529/comments","events_url":"https://api.github.com/repos/balloob/home-assistant/issues/1529/events","html_url":"https://github.com/balloob/home-assistant/pull/1529","id":140418246,"number":1529,"title":"Rewrite of the tellstick module. It now uses a common base for all sh…","user":{"login":"stefan-jonasson","id":1372444,"avatar_url":"https://avatars.githubusercontent.com/u/1372444?v=3","gravatar_id":"","url":"https://api.github.com/users/stefan-jonasson","html_url":"https://github.com/stefan-jonasson","followers_url":"https://api.github.com/users/stefan-jonasson/followers","following_url":"https://api.github.com/users/stefan-jonasson/following{/other_user}","gists_url":"https://api.github.com/users/stefan-jonasson/gists{/gist_id}","starred_url":"https://api.github.com/users/stefan-jonasson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/stefan-jonasson/subscriptions","organizations_url":"https://api.github.com/users/stefan-jonasson/orgs","repos_url":"https://api.github.com/users/stefan-jonasson/repos","events_url":"https://api.github.com/users/stefan-jonasson/events{/privacy}","received_events_url":"https://api.github.com/users/stefan-jonasson/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2016-03-12T21:28:17Z","updated_at":"2016-03-13T16:28:34Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/balloob/home-assistant/pulls/1529","html_url":"https://github.com/balloob/home-assistant/pull/1529","diff_url":"https://github.com/balloob/home-assistant/pull/1529.diff","patch_url":"https://github.com/balloob/home-assistant/pull/1529.patch"},"body":"**Description:**\r\nRewrite of the tellstick module. It now uses a common base for all shared functionality.\r\n\r\nThe rewrite addresses a problem with the tellstick hardware dropping commands when too many simultaneous calls is being made from HA. \r\nAlso fixes a bug when the dim level was changed externally.\r\nThis breaks previous configurations.\r\n\r\n**Example entry for `configuration.yaml`**\r\nThe new config for tellstick is\r\n```yaml\r\ntellstick:\r\n  signal_repetitions: X\r\n```\r\nLights and Switches are detected automatically.\r\nSensors work like before because they do not share any functionality with the other devices and they also needs a complete other configuration.\r\n\r\nTODO:\r\nThe telldus core support devices that take UP,DOWN and STOP commands as well, these are not mapped to any home assistant component. I don't have any of these devices so implementing them would be difficult. Also I don't know what component to map them to. Maybe we could have something similar to binary_sensor but for things that goes up and down?\r\n\r\n**Related issue (if applicable):** \r\n #422\r\n\r\n**Checklist:**\r\n\r\n- [x] Local tests with `tox` run successfully.\r\n- [x] TravisCI does not fail. **Your PR cannot be merged unless CI is green!**\r\n- [x] [Fork is up to date][fork] and was rebased on the `dev` branch before creating the PR.\r\n- [x] Commits have been [squashed][squash].\r\n  - [x] New files were added to `.coveragerc`.\r\n\r\n[fork]: http://stackoverflow.com/a/7244456\r\n[squash]: https://github.com/ginatrapani/todo.txt-android/wiki/Squash-All-Commits-Related-to-a-Single-Issue-into-a-Single-Commit\r\n[ex-requir]: https://github.com/balloob/home-assistant/blob/dev/homeassistant/components/keyboard.py#L16\r\n[ex-import]: https://github.com/balloob/home-assistant/blob/dev/homeassistant/components/keyboard.py#L51\r\n\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/balloob/home-assistant/issues/comments/195989863","html_url":"https://github.com/balloob/home-assistant/pull/1529#issuecomment-195989863","issue_url":"https://api.github.com/repos/balloob/home-assistant/issues/1529","id":195989863,"user":{"login":"stefan-jonasson","id":1372444,"avatar_url":"https://avatars.githubusercontent.com/u/1372444?v=3","gravatar_id":"","url":"https://api.github.com/users/stefan-jonasson","html_url":"https://github.com/stefan-jonasson","followers_url":"https://api.github.com/users/stefan-jonasson/followers","following_url":"https://api.github.com/users/stefan-jonasson/following{/other_user}","gists_url":"https://api.github.com/users/stefan-jonasson/gists{/gist_id}","starred_url":"https://api.github.com/users/stefan-jonasson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/stefan-jonasson/subscriptions","organizations_url":"https://api.github.com/users/stefan-jonasson/orgs","repos_url":"https://api.github.com/users/stefan-jonasson/repos","events_url":"https://api.github.com/users/stefan-jonasson/events{/privacy}","received_events_url":"https://api.github.com/users/stefan-jonasson/received_events","type":"User","site_admin":false},"created_at":"2016-03-13T16:28:34Z","updated_at":"2016-03-13T16:28:34Z","body":"The `None` for brightness is already fixed, your comment was on the wrong line. \r\n\r\nAbout sensors, as I said in the description, they are intentionally left out of this rewrite because they do not share any functionality. Do you think it would be less confusing for the user if they was merged as well? "}}