{"action":"created","issue":{"url":"https://api.github.com/repos/pelias/api/issues/121","labels_url":"https://api.github.com/repos/pelias/api/issues/121/labels{/name}","comments_url":"https://api.github.com/repos/pelias/api/issues/121/comments","events_url":"https://api.github.com/repos/pelias/api/issues/121/events","html_url":"https://github.com/pelias/api/pull/121","id":73099960,"number":121,"title":"wrap longitude (#56)","user":{"login":"sevko","id":4467604,"avatar_url":"https://avatars.githubusercontent.com/u/4467604?v=3","gravatar_id":"","url":"https://api.github.com/users/sevko","html_url":"https://github.com/sevko","followers_url":"https://api.github.com/users/sevko/followers","following_url":"https://api.github.com/users/sevko/following{/other_user}","gists_url":"https://api.github.com/users/sevko/gists{/gist_id}","starred_url":"https://api.github.com/users/sevko/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sevko/subscriptions","organizations_url":"https://api.github.com/users/sevko/orgs","repos_url":"https://api.github.com/users/sevko/repos","events_url":"https://api.github.com/users/sevko/events{/privacy}","received_events_url":"https://api.github.com/users/sevko/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/pelias/api/labels/in%20progress","name":"in progress","color":"ededed"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-05-04T19:04:15Z","updated_at":"2015-05-04T21:25:12Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/pelias/api/pulls/121","html_url":"https://github.com/pelias/api/pull/121","diff_url":"https://github.com/pelias/api/pull/121.diff","patch_url":"https://github.com/pelias/api/pull/121.patch"},"body":"Implement longitude wrapping by removing the longitude sanitizer (elasticsearch appears to gracefully handle lat/lon wrapping, such that it treats a longitude of 181 as -179). Note that while I ripped out some of the bits in `sanitizer/_geo`, I simultaneously refactored some convoluted code, so there are some slightly unrelated changes.\r\n\r\nfixes pelias/api#56"},"comment":{"url":"https://api.github.com/repos/pelias/api/issues/comments/98853298","html_url":"https://github.com/pelias/api/pull/121#issuecomment-98853298","issue_url":"https://api.github.com/repos/pelias/api/issues/121","id":98853298,"user":{"login":"dianashk","id":5446511,"avatar_url":"https://avatars.githubusercontent.com/u/5446511?v=3","gravatar_id":"","url":"https://api.github.com/users/dianashk","html_url":"https://github.com/dianashk","followers_url":"https://api.github.com/users/dianashk/followers","following_url":"https://api.github.com/users/dianashk/following{/other_user}","gists_url":"https://api.github.com/users/dianashk/gists{/gist_id}","starred_url":"https://api.github.com/users/dianashk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dianashk/subscriptions","organizations_url":"https://api.github.com/users/dianashk/orgs","repos_url":"https://api.github.com/users/dianashk/repos","events_url":"https://api.github.com/users/dianashk/events{/privacy}","received_events_url":"https://api.github.com/users/dianashk/received_events","type":"User","site_admin":false},"created_at":"2015-05-04T21:25:11Z","updated_at":"2015-05-04T21:25:11Z","body":"If ES handles both lat and lon, why did you only remove the lon sanitizer? That seems odd to me. We should either remove both or sanitize both.\r\n\r\nAlso please add acceptance tests for this. Thanks."}}