{"action":"created","issue":{"url":"https://api.github.com/repos/CartoDB/cartodb/issues/12086","repository_url":"https://api.github.com/repos/CartoDB/cartodb","labels_url":"https://api.github.com/repos/CartoDB/cartodb/issues/12086/labels{/name}","comments_url":"https://api.github.com/repos/CartoDB/cartodb/issues/12086/comments","events_url":"https://api.github.com/repos/CartoDB/cartodb/issues/12086/events","html_url":"https://github.com/CartoDB/cartodb/pull/12086","id":226039055,"number":12086,"title":"It is not necessary to trigger fetchs for reset-style-per-node when node itself launches one when it is ready/finished","user":{"login":"xavijam","id":132146,"avatar_url":"https://avatars0.githubusercontent.com/u/132146?v=3","gravatar_id":"","url":"https://api.github.com/users/xavijam","html_url":"https://github.com/xavijam","followers_url":"https://api.github.com/users/xavijam/followers","following_url":"https://api.github.com/users/xavijam/following{/other_user}","gists_url":"https://api.github.com/users/xavijam/gists{/gist_id}","starred_url":"https://api.github.com/users/xavijam/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/xavijam/subscriptions","organizations_url":"https://api.github.com/users/xavijam/orgs","repos_url":"https://api.github.com/users/xavijam/repos","events_url":"https://api.github.com/users/xavijam/events{/privacy}","received_events_url":"https://api.github.com/users/xavijam/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2017-05-03T16:13:50Z","updated_at":"2017-05-05T10:25:28Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/CartoDB/cartodb/pulls/12086","html_url":"https://github.com/CartoDB/cartodb/pull/12086","diff_url":"https://github.com/CartoDB/cartodb/pull/12086.diff","patch_url":"https://github.com/CartoDB/cartodb/pull/12086.patch"},"body":"Related ticket: https://github.com/CartoDB/cartodb/issues/12085\r\n\r\nWhen we introduced https://github.com/CartoDB/cartodb/pull/11989, we didn't realize we f***** up reset-styles thing. Because it was resetting the status of the query objects from the deep-insights-integration node changes (https://github.com/CartoDB/cartodb/blob/master/lib/assets/core/javascripts/cartodb3/deep-insights-integrations.js#L347-L375).\r\n\r\nIn the future, we will remove this and we will apply the specific style for each type of analysis (cc @saleiva @makella).\r\n\r\nIn order to test this, I'd check that when a new analysis is added, reset styles works properly. Visit styles panel and check that the simple one is selected when the analysis is finished."},"comment":{"url":"https://api.github.com/repos/CartoDB/cartodb/issues/comments/299431655","html_url":"https://github.com/CartoDB/cartodb/pull/12086#issuecomment-299431655","issue_url":"https://api.github.com/repos/CartoDB/cartodb/issues/12086","id":299431655,"user":{"login":"javitonino","id":7569673,"avatar_url":"https://avatars0.githubusercontent.com/u/7569673?v=3","gravatar_id":"","url":"https://api.github.com/users/javitonino","html_url":"https://github.com/javitonino","followers_url":"https://api.github.com/users/javitonino/followers","following_url":"https://api.github.com/users/javitonino/following{/other_user}","gists_url":"https://api.github.com/users/javitonino/gists{/gist_id}","starred_url":"https://api.github.com/users/javitonino/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/javitonino/subscriptions","organizations_url":"https://api.github.com/users/javitonino/orgs","repos_url":"https://api.github.com/users/javitonino/repos","events_url":"https://api.github.com/users/javitonino/events{/privacy}","received_events_url":"https://api.github.com/users/javitonino/received_events","type":"User","site_admin":false},"created_at":"2017-05-05T10:25:28Z","updated_at":"2017-05-05T10:25:28Z","body":"I have been able to reproduce with a bigger dataset and a more complex analysis. Continuing..., thanks!"}}
{"actor":{"display_login":"javitonino"}}