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

[9.0][PORT] base_partner_merge #274

Merged
merged 2 commits into from
Jun 16, 2016

Conversation

yvaucher
Copy link
Member

Migration of module base_partner_merge.

It proposes a simple way allows to use the crm deduplicate wizard without having to install CRM module.

See the discution here: #196 (comment)

Thanks @hbrunn for the idea.

I set _module = 'base_partner_merge' to ensure the model are loaded.

I will let you do other PRs to extend or modify the behaviour of the wizard if needed to alter the way the official wizard works.

'description': """
backport module, to be removed when we switch to saas2 on the private servers
""",
'version': '9.0.0.1.0',
Copy link
Member

Choose a reason for hiding this comment

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

9.0.1.0.0

@coveralls
Copy link

coveralls commented May 13, 2016

Coverage Status

Changes Unknown when pulling ba0e50d on yvaucher:9.0-port-base_merge_partner into * on OCA:9.0*.

@pedrobaeza
Copy link
Member

Please bump the version and check Travis.

@yvaucher yvaucher force-pushed the 9.0-port-base_merge_partner branch from ba0e50d to b0244c2 Compare May 31, 2016 11:36
@yvaucher yvaucher force-pushed the 9.0-port-base_merge_partner branch from b0244c2 to a2fef93 Compare May 31, 2016 11:37
@yvaucher
Copy link
Member Author

Fixed

@hbrunn
Copy link
Member

hbrunn commented May 31, 2016

cool that this works! I pondered if it's a good idea to load the view from the crm addon to in a function called from xml files, but that's a bit tricky and we wouldn't be able to add extra features, so I think this is fine the way it is 👍

@hbrunn hbrunn modified the milestones: 8.0, 9.0 May 31, 2016
@coveralls
Copy link

coveralls commented May 31, 2016

Coverage Status

Changes Unknown when pulling a2fef93 on yvaucher:9.0-port-base_merge_partner into * on OCA:9.0*.

@lmignon
Copy link
Contributor

lmignon commented Jun 16, 2016

👍 Works like a charm. Thanh you @pedrobaeza

@pedrobaeza pedrobaeza merged commit 2c202f9 into OCA:9.0 Jun 16, 2016
@yvaucher
Copy link
Member Author

@lmignon wrong PR? Or didn't got your tee/coffee yet? 😄

@yvaucher
Copy link
Member Author

Thanks for the merge @pedrobaeza

@lmignon
Copy link
Contributor

lmignon commented Jun 16, 2016

oupssss not the wrong PR my thankfulness is for you @yvaucher

JayVora-SerpentCS pushed a commit to JayVora-SerpentCS/partner-contact that referenced this pull request Jan 9, 2018
[9.0][PORT] base_partner_merge
JayVora-SerpentCS pushed a commit to JayVora-SerpentCS/partner-contact that referenced this pull request Jan 9, 2018
[9.0][PORT] base_partner_merge
JayVora-SerpentCS pushed a commit to JayVora-SerpentCS/partner-contact that referenced this pull request Jan 9, 2018
[9.0][PORT] base_partner_merge
yvaucher added a commit that referenced this pull request Jun 28, 2018
[9.0][PORT] base_partner_merge
anandkansagra pushed a commit to anandkansagra/partner-contact that referenced this pull request Dec 31, 2018
[9.0][PORT] base_partner_merge
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants