Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FIX] allow to change partner_id_display #84

Merged
merged 4 commits into from
Feb 6, 2015

Conversation

hbrunn
Copy link
Member

@hbrunn hbrunn commented Jan 26, 2015

No description provided.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.11%) to 47.06% when pulling 18c5d0c on hbrunn:7.0-partner_relations into b63c7aa on OCA:7.0.

@pedrobaeza
Copy link
Member

👍

@hbrunn
Copy link
Member Author

hbrunn commented Jan 28, 2015

I added a commit that also fixes a logic error

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.11%) to 47.06% when pulling a8d0f4b on hbrunn:7.0-partner_relations into b63c7aa on OCA:7.0.

@guewen
Copy link
Member

guewen commented Jan 29, 2015

👍

@hbrunn
Copy link
Member Author

hbrunn commented Jan 29, 2015

and I added another commit that allows searching for translated relation names (res.partner.relation.type selection is a view on res.partner.relation.type, so we don't get openerp's translation mechanism on that model)

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.11%) to 47.06% when pulling c0e19e4 on hbrunn:7.0-partner_relations into b63c7aa on OCA:7.0.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.14%) to 47.02% when pulling edca46c on hbrunn:7.0-partner_relations into b63c7aa on OCA:7.0.

@yvaucher
Copy link
Member

yvaucher commented Feb 6, 2015

👍

yvaucher added a commit that referenced this pull request Feb 6, 2015
[FIX] allow to change partner_id_display
@yvaucher yvaucher merged commit 98b825a into OCA:7.0 Feb 6, 2015
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.

None yet

5 participants