{"action":"closed","issue":{"url":"https://api.github.com/repos/CartoDB/cartodb/issues/10688","repository_url":"https://api.github.com/repos/CartoDB/cartodb","labels_url":"https://api.github.com/repos/CartoDB/cartodb/issues/10688/labels{/name}","comments_url":"https://api.github.com/repos/CartoDB/cartodb/issues/10688/comments","events_url":"https://api.github.com/repos/CartoDB/cartodb/issues/10688/events","html_url":"https://github.com/CartoDB/cartodb/issues/10688","id":188530172,"number":10688,"title":"Builder fails if migrated visualization contains a layer with null tooltip","user":{"login":"juanignaciosl","id":932968,"avatar_url":"https://avatars.githubusercontent.com/u/932968?v=3","gravatar_id":"","url":"https://api.github.com/users/juanignaciosl","html_url":"https://github.com/juanignaciosl","followers_url":"https://api.github.com/users/juanignaciosl/followers","following_url":"https://api.github.com/users/juanignaciosl/following{/other_user}","gists_url":"https://api.github.com/users/juanignaciosl/gists{/gist_id}","starred_url":"https://api.github.com/users/juanignaciosl/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/juanignaciosl/subscriptions","organizations_url":"https://api.github.com/users/juanignaciosl/orgs","repos_url":"https://api.github.com/users/juanignaciosl/repos","events_url":"https://api.github.com/users/juanignaciosl/events{/privacy}","received_events_url":"https://api.github.com/users/juanignaciosl/received_events","type":"User","site_admin":false},"labels":[{"id":8530709,"url":"https://api.github.com/repos/CartoDB/cartodb/labels/bug","name":"bug","color":"fc2929","default":true},{"id":313918117,"url":"https://api.github.com/repos/CartoDB/cartodb/labels/Builder","name":"Builder","color":"207de5","default":false}],"state":"closed","locked":false,"assignee":{"login":"juanignaciosl","id":932968,"avatar_url":"https://avatars.githubusercontent.com/u/932968?v=3","gravatar_id":"","url":"https://api.github.com/users/juanignaciosl","html_url":"https://github.com/juanignaciosl","followers_url":"https://api.github.com/users/juanignaciosl/followers","following_url":"https://api.github.com/users/juanignaciosl/following{/other_user}","gists_url":"https://api.github.com/users/juanignaciosl/gists{/gist_id}","starred_url":"https://api.github.com/users/juanignaciosl/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/juanignaciosl/subscriptions","organizations_url":"https://api.github.com/users/juanignaciosl/orgs","repos_url":"https://api.github.com/users/juanignaciosl/repos","events_url":"https://api.github.com/users/juanignaciosl/events{/privacy}","received_events_url":"https://api.github.com/users/juanignaciosl/received_events","type":"User","site_admin":false},"assignees":[{"login":"juanignaciosl","id":932968,"avatar_url":"https://avatars.githubusercontent.com/u/932968?v=3","gravatar_id":"","url":"https://api.github.com/users/juanignaciosl","html_url":"https://github.com/juanignaciosl","followers_url":"https://api.github.com/users/juanignaciosl/followers","following_url":"https://api.github.com/users/juanignaciosl/following{/other_user}","gists_url":"https://api.github.com/users/juanignaciosl/gists{/gist_id}","starred_url":"https://api.github.com/users/juanignaciosl/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/juanignaciosl/subscriptions","organizations_url":"https://api.github.com/users/juanignaciosl/orgs","repos_url":"https://api.github.com/users/juanignaciosl/repos","events_url":"https://api.github.com/users/juanignaciosl/events{/privacy}","received_events_url":"https://api.github.com/users/juanignaciosl/received_events","type":"User","site_admin":false}],"milestone":{"url":"https://api.github.com/repos/CartoDB/cartodb/milestones/121","html_url":"https://github.com/CartoDB/cartodb/milestones/Builder%20-%20La%20Guardia","labels_url":"https://api.github.com/repos/CartoDB/cartodb/milestones/121/labels","id":2116514,"number":121,"title":"Builder - La Guardia","description":"Now:\r\n\r\n- Ghost nodes bugfixing (@guido) *.\r\n- General bugfixing (@xavijam @hacheka @mariacheca) *.\r\n- On premise problems and additions (@xavijam and any backender if needed) *.\r\n- Implement flatten analysis layer (@mariacheca).\r\n- Style by icon functionality (FG) (@javierarce).\r\n- Zoom to data (@hacheka).\r\n- Finish Geometry Edition enhancements (@matallo).\r\n- Finish legends bugfixing (@nobuti).\r\n- Finish auto-style fixes and enhancements for version 0.2 (@nobuti + @xavijam).\r\n- Pricing changes in upgrade user (@mariacheca).\r\n\r\nNext: \r\n\r\n- Models refactor (@CartoDB/builder-backend).\r\n\r\n\r\n*: Things those should go in the new onpremise version.\r\nFG: we encourage to implement it with a feature flag.","creator":{"login":"saleiva","id":132149,"avatar_url":"https://avatars.githubusercontent.com/u/132149?v=3","gravatar_id":"","url":"https://api.github.com/users/saleiva","html_url":"https://github.com/saleiva","followers_url":"https://api.github.com/users/saleiva/followers","following_url":"https://api.github.com/users/saleiva/following{/other_user}","gists_url":"https://api.github.com/users/saleiva/gists{/gist_id}","starred_url":"https://api.github.com/users/saleiva/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/saleiva/subscriptions","organizations_url":"https://api.github.com/users/saleiva/orgs","repos_url":"https://api.github.com/users/saleiva/repos","events_url":"https://api.github.com/users/saleiva/events{/privacy}","received_events_url":"https://api.github.com/users/saleiva/received_events","type":"User","site_admin":false},"open_issues":30,"closed_issues":27,"state":"open","created_at":"2016-11-05T16:22:03Z","updated_at":"2016-11-24T09:26:28Z","due_on":"2016-12-01T00:00:00Z","closed_at":null},"comments":1,"created_at":"2016-11-10T14:59:46Z","updated_at":"2016-11-24T09:26:28Z","closed_at":"2016-11-24T09:26:28Z","body":"### Context\r\n* Reported by a customer.\r\n\r\n### Steps to Reproduce\r\nWe haven't been able to reproduce: layers should always contain a valid, nonempty tooltip.\r\n\r\nLaboratory test:\r\n1. Create a visualization with the editor.\r\n2. `update layers set tooltip = null where id = '85f5f10d-8874-4899-b61a-7d0a8ffbe1e0';`\r\n3. Go to popup option at the builder.\r\n\r\nEmpty tooltip is valid for non-data layers, forcing the migration to set a default value if the layer is a data one would fix this for non-migrated visualizations, but maybe this must be handled as well in fronted.\r\n\r\nIf anybody is able to reproduce this, please let me know\r\n\r\ncc @xavijam @CartoDB/builder-backend @CartoDB/frontend "}}
{"actor":{"display_login":"juanignaciosl"}}