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

[12.0][MIG] partner_multi_relation_tabs #863

Merged
merged 8 commits into from Mar 18, 2020

Conversation

janverb
Copy link
Contributor

@janverb janverb commented Mar 4, 2020

No description provided.

@lfreeke
Copy link
Contributor

lfreeke commented Mar 5, 2020

It seems we don't have runbot here @NL66278 Can you do anything about this?

@NL66278
Copy link
Contributor

NL66278 commented Mar 5, 2020

@lfreeke I have no idea what I could do about this.

Copy link
Contributor

@NL66278 NL66278 left a comment

Choose a reason for hiding this comment

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

👍 LGTM

NL66278 and others added 8 commits March 9, 2020 15:03
- Separate model (db) partially from user interface;
- Use demo data for performance, cleaner tests, and front end testing;
- Make sure tabs dependent on partner_category_id work.
Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: partner-contact-11.0/partner-contact-11.0-partner_multi_relation_tabs
Translate-URL: https://translation.odoo-community.org/projects/partner-contact-11-0/partner-contact-11-0-partner_multi_relation_tabs/

Added translation using Weblate (Danish)

Translated using Weblate (Danish)

Currently translated at 10.4% (5 of 48 strings)

Translation: partner-contact-11.0/partner-contact-11.0-partner_multi_relation_tabs
Translate-URL: https://translation.odoo-community.org/projects/partner-contact-11-0/partner-contact-11-0-partner_multi_relation_tabs/da/
@janverb janverb force-pushed the 12.0-mig-partner_multi_relation_tabs branch from f4ec588 to 5c37952 Compare March 9, 2020 14:06
Copy link
Contributor

@lfreeke lfreeke left a comment

Choose a reason for hiding this comment

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

Functional test 👍

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@NL66278
Copy link
Contributor

NL66278 commented Mar 11, 2020

/ocabot merge

@NL66278
Copy link
Contributor

NL66278 commented Mar 18, 2020

/ocabot merge

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 12.0-ocabot-merge-pr-863-by-NL66278-bump-no, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit be0aa4e into OCA:12.0 Mar 18, 2020
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 036f6d9. Thanks a lot for contributing to OCA. ❤️

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.

None yet

6 participants