Skip to content

[Add] Display labels on polygons#162

Merged
yvaucher merged 3 commits into
OCA:10.0from
dek-odoo:10.0
Jun 28, 2018
Merged

[Add] Display labels on polygons#162
yvaucher merged 3 commits into
OCA:10.0from
dek-odoo:10.0

Conversation

@dek-odoo
Copy link
Copy Markdown
Contributor

@dek-odoo dek-odoo commented Apr 1, 2018

Description

Now user can see what each polygon is about.

zip polygons with labels

Configuration:

  • Need to tick "Display Labels on Polygon" field
  • Select appropriate attribute field

configuration

Pleasure to help, changes are welcome.
Demo video.

@dek-odoo dek-odoo changed the title [Add] Allow to display labels on polygons for layers [Add] Display labels on polygons Apr 1, 2018
@dek-odoo
Copy link
Copy Markdown
Contributor Author

dek-odoo commented Apr 8, 2018

@yvaucher @lmignon

Copy link
Copy Markdown
Member

@yvaucher yvaucher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice addition, LGTM

@dek-odoo dek-odoo force-pushed the 10.0 branch 2 times, most recently from 984a496 to 9609ef8 Compare June 2, 2018 18:37
Copy link
Copy Markdown
Contributor

@lmignon lmignon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but we must fix pylint before merging

@dek-odoo
Copy link
Copy Markdown
Contributor Author

dek-odoo commented Jun 16, 2018

Hi @lmignon
The Travis CI issues do not seem to be from my code.
Similar to here

@yvaucher yvaucher merged commit 8e53db9 into OCA:10.0 Jun 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants