{"action":"created","issue":{"url":"https://api.github.com/repos/CartoDB/cartodb/issues/11962","repository_url":"https://api.github.com/repos/CartoDB/cartodb","labels_url":"https://api.github.com/repos/CartoDB/cartodb/issues/11962/labels{/name}","comments_url":"https://api.github.com/repos/CartoDB/cartodb/issues/11962/comments","events_url":"https://api.github.com/repos/CartoDB/cartodb/issues/11962/events","html_url":"https://github.com/CartoDB/cartodb/issues/11962","id":220975289,"number":11962,"title":"Make changes in the assets build process","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":[{"id":142731156,"url":"https://api.github.com/repos/CartoDB/cartodb/labels/Frontend","name":"Frontend","color":"fbca04","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/CartoDB/cartodb/milestones/139","html_url":"https://github.com/CartoDB/cartodb/milestone/139","labels_url":"https://api.github.com/repos/CartoDB/cartodb/milestones/139/labels","id":2413986,"number":139,"title":"Frontend bundle assets improvements","description":"- Improve test bundle and watch.\r\n- Improve dev assets bundle and watch.\r\n- Replace `npm` by `yarn`.\r\n- Split frontend from backend.","creator":{"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},"open_issues":3,"closed_issues":2,"state":"open","created_at":"2017-03-27T15:03:33Z","updated_at":"2017-05-24T17:45:00Z","due_on":null,"closed_at":null},"comments":19,"created_at":"2017-04-11T14:42:37Z","updated_at":"2017-05-25T10:25:35Z","closed_at":null,"body":"#What does it mean? Basically improve the way we prepare the assets for development or production. \r\nThe main purposes are:\r\n\r\n- Improve development speed: watch, bundle, initialization,...\r\n- Improve deployment speed: don't uglify already minified vendor files, for example, don't run unnecessary development tasks here, etc.\r\n\r\nIt has a lot of implications, but the main tasks should be:\r\n\r\n## Main tasks\r\n- [x] Migrate from Browserify to **Webpack**: already done in the test part thanks to @ivanmalagon.\r\n- [x] Taking into account the previous bullet, we should not migrate the old code (editor) to this new \"workflow\". That means only, for the moment, **builder**, **dataset** and **public** embed views will be \"migrated\".\r\n- [x] Start using **lib-sass** for compiling the CSS instead of **~compass~** for the whole project. We should leave this last one only for the old assets.\r\n- [ ] We should check if new tests task needs to be revisited after those new changes.\r\n\r\n## Caveats\r\n- [x] We need to keep using the **core <> client** behaviour for assets touched \"outside\" of our usual workflow (mainly for solutions engineers).\r\n- [ ] Vendor files (like tangram) don't need to be uglified, we should avoid this in order to gain time in the deploys.\r\n- [x] When a new file is added, it should be taken by the watch process.\r\n\r\n## Improvements\r\n- [x] Research how to improve javascript source-map support for staging and production.\r\n\r\n## Nice to have\r\n- [ ] Hot module replacement\r\n- [ ] Smart build: only build touched code. If the edited/new code belongs to Builder, only build the Builder. If the code belongs to Editor, only build the Editor. If there is code for both sides, build Builder and Editor.\r\n\r\n@CartoDB/frontend "},"comment":{"url":"https://api.github.com/repos/CartoDB/cartodb/issues/comments/303977420","html_url":"https://github.com/CartoDB/cartodb/issues/11962#issuecomment-303977420","issue_url":"https://api.github.com/repos/CartoDB/cartodb/issues/11962","id":303977420,"user":{"login":"matallo","id":36676,"avatar_url":"https://avatars1.githubusercontent.com/u/36676?v=3","gravatar_id":"","url":"https://api.github.com/users/matallo","html_url":"https://github.com/matallo","followers_url":"https://api.github.com/users/matallo/followers","following_url":"https://api.github.com/users/matallo/following{/other_user}","gists_url":"https://api.github.com/users/matallo/gists{/gist_id}","starred_url":"https://api.github.com/users/matallo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/matallo/subscriptions","organizations_url":"https://api.github.com/users/matallo/orgs","repos_url":"https://api.github.com/users/matallo/repos","events_url":"https://api.github.com/users/matallo/events{/privacy}","received_events_url":"https://api.github.com/users/matallo/received_events","type":"User","site_admin":false},"created_at":"2017-05-25T10:25:35Z","updated_at":"2017-05-25T10:25:35Z","body":"btw, I'm updating this with last changes in https://github.com/CartoDB/cartodb/pull/12203"}}
{"actor":{"display_login":"matallo"}}