Code review comment for lp:~openerp-community/geospatial-addons/7.0
Revision history for this message
| Nicolas Bessi - Camptocamp (nbessi-c2c-deactivatedaccount) wrote | # |
review:
Needs Information
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
0e1f616
(Get the code!)
Hello,
Thanks for your MP. Fixing view can be quite a pain…
I have fixed some code in geo sale module to avoid installation error.
That said I have also set module to be uninstallable.
The map view (outside of form) is broken there is some oddities in edit mode.
There is no test at that time (All what is needed is available but not in the correct form) and other strange behavior to be fixed with attrs in layers view.
At that time addon does not reach community standard to be merges as is. So having addons uninstallable will allows me to merge you work as a progression in the port of the modules.
If you agree with that i will approve your MP.
I know it is quite a huge work to port this module. Thanks for the work done.
Regards
Nicolas