{"action":"created","issue":{"url":"https://api.github.com/repos/OCA/geospatial/issues/24","labels_url":"https://api.github.com/repos/OCA/geospatial/issues/24/labels{/name}","comments_url":"https://api.github.com/repos/OCA/geospatial/issues/24/comments","events_url":"https://api.github.com/repos/OCA/geospatial/issues/24/events","html_url":"https://github.com/OCA/geospatial/pull/24","id":60099536,"number":24,"title":"8.0 migrate from 7.0","user":{"login":"lmignon","id":544090,"avatar_url":"https://avatars.githubusercontent.com/u/544090?v=3","gravatar_id":"","url":"https://api.github.com/users/lmignon","html_url":"https://github.com/lmignon","followers_url":"https://api.github.com/users/lmignon/followers","following_url":"https://api.github.com/users/lmignon/following{/other_user}","gists_url":"https://api.github.com/users/lmignon/gists{/gist_id}","starred_url":"https://api.github.com/users/lmignon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lmignon/subscriptions","organizations_url":"https://api.github.com/users/lmignon/orgs","repos_url":"https://api.github.com/users/lmignon/repos","events_url":"https://api.github.com/users/lmignon/events{/privacy}","received_events_url":"https://api.github.com/users/lmignon/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/OCA/geospatial/labels/needs%20fixing","name":"needs fixing","color":"eb6420"},{"url":"https://api.github.com/repos/OCA/geospatial/labels/needs%20review","name":"needs review","color":"fbca04"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/OCA/geospatial/milestones/3","html_url":"https://github.com/OCA/geospatial/milestones/8.0","labels_url":"https://api.github.com/repos/OCA/geospatial/milestones/3/labels","id":988414,"number":3,"title":"8.0","description":null,"creator":{"login":"bwrsandman","id":1013356,"avatar_url":"https://avatars.githubusercontent.com/u/1013356?v=3","gravatar_id":"","url":"https://api.github.com/users/bwrsandman","html_url":"https://github.com/bwrsandman","followers_url":"https://api.github.com/users/bwrsandman/followers","following_url":"https://api.github.com/users/bwrsandman/following{/other_user}","gists_url":"https://api.github.com/users/bwrsandman/gists{/gist_id}","starred_url":"https://api.github.com/users/bwrsandman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bwrsandman/subscriptions","organizations_url":"https://api.github.com/users/bwrsandman/orgs","repos_url":"https://api.github.com/users/bwrsandman/repos","events_url":"https://api.github.com/users/bwrsandman/events{/privacy}","received_events_url":"https://api.github.com/users/bwrsandman/received_events","type":"User","site_admin":false},"open_issues":1,"closed_issues":1,"state":"open","created_at":"2015-02-20T15:13:50Z","updated_at":"2015-03-20T17:17:47Z","due_on":null,"closed_at":null},"comments":28,"created_at":"2015-03-06T13:07:39Z","updated_at":"2015-05-08T12:53:40Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/OCA/geospatial/pulls/24","html_url":"https://github.com/OCA/geospatial/pull/24","diff_url":"https://github.com/OCA/geospatial/pull/24.diff","patch_url":"https://github.com/OCA/geospatial/pull/24.patch"},"body":"Current status:\r\n\r\n- [x] Forward port changes from PR #16 \r\n- [x] Defines Fields for specialized geo columns \r\n- [x] Migrate to the new API\r\n- [x] Extract description in \\__openerp\\__.py to README.rst\r\n- [x] Use an init hook to initialize postgis extension at install\r\n- [x] Move icons to static/description\r\n- [x] Fix issues when google is used as Raster Layer Type (https://github.com/acsone/geospatial/commit/6f924a2d7df99a7c956851ddd499fdb6db22a4f4 should be backported in 7.0)\r\n- [x] ~~Fix issues when the map is displayed for a specific field. (the label is not visible the map takes the whole width)~~ The view definition in the demo data was wrong. (see https://github.com/acsone/geospatial/commit/9d939da17211b8ff1e787b3c6475fbbae5e1bc01)\r\n\r\nAt this stage addons are usable in V8 but some remaining issues are to be fixed. However, two bugs in Odoo prevent proper use. To work, you need to merge the following 2 branches.\r\n-   https://github.com/acsone/odoo/tree/8.0-fix-5616-ir_model_fields-from-fields (PR odoo/odoo/pull/5618)\r\n-  https://github.com/acsone/odoo/tree/8.0-fix-use_symbol-c-in-sql-create (PR odoo/odoo/pull/5620)"},"comment":{"url":"https://api.github.com/repos/OCA/geospatial/issues/comments/100224087","html_url":"https://github.com/OCA/geospatial/pull/24#issuecomment-100224087","issue_url":"https://api.github.com/repos/OCA/geospatial/issues/24","id":100224087,"user":{"login":"nbessi","id":147671,"avatar_url":"https://avatars.githubusercontent.com/u/147671?v=3","gravatar_id":"","url":"https://api.github.com/users/nbessi","html_url":"https://github.com/nbessi","followers_url":"https://api.github.com/users/nbessi/followers","following_url":"https://api.github.com/users/nbessi/following{/other_user}","gists_url":"https://api.github.com/users/nbessi/gists{/gist_id}","starred_url":"https://api.github.com/users/nbessi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nbessi/subscriptions","organizations_url":"https://api.github.com/users/nbessi/orgs","repos_url":"https://api.github.com/users/nbessi/repos","events_url":"https://api.github.com/users/nbessi/events{/privacy}","received_events_url":"https://api.github.com/users/nbessi/received_events","type":"User","site_admin":false},"created_at":"2015-05-08T12:53:40Z","updated_at":"2015-05-08T12:53:40Z","body":"@lmignon I have started to work on your work. We have a nasty situation here. The initialisation of the colums is made with old api model, but fields can be of new api type etc. I propose to transfer all the know how to the new api fields and to base all the initialization of fields base on the _fields property.\r\n\r\nI also propose to implement a dedicated _field_create for geofields that shoud be clean and fast.\r\n\r\nI also really hesitate to keep old style field. At least it should only contains read write functionality but initialization should always been delegated to new style fields, it should not be to complicated.\r\n\r\nAlso the new style fields do not seem to be called. I have to find why...\r\n\r\n"}}