Skip to content

[13.0] MIG base_contract#578

Closed
alexis-via wants to merge 23 commits intoOCA:13.0from
akretion:13-mig-agreement-travis-green2
Closed

[13.0] MIG base_contract#578
alexis-via wants to merge 23 commits intoOCA:13.0from
akretion:13-mig-agreement-travis-green2

Conversation

@alexis-via
Copy link
Copy Markdown
Contributor

Superseeds PR #397 (I needed to do a rebase).

ygol and others added 23 commits November 19, 2020 16:39
Currently translated at 100.0% (27 of 27 strings)

Translation: contract-12.0/contract-12.0-agreement
Translate-URL: https://translation.odoo-community.org/projects/contract-12-0/contract-12-0-agreement/pt_PT/
we move the is_template field definition and the agreement.type model from
the agreement_legal module to the agreement module.

The fields are not displayed by default, unless the feature is enabled through a
technical feature group, this is configurable in the agreement_sale module (because agreement
in itself has no UI, and agreement_legal enables the feature by default)
Replace deprecated _company_default_get() by standard v13 syntax
Add domain field
Add chatter on agreement object
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Hi @ygol,
some modules you are maintaining are being modified, check this out!

@sergio-teruel
Copy link
Copy Markdown
Contributor

@alexis-via Can you add this commit from v12.0?
b61a0cc

@alexis-via
Copy link
Copy Markdown
Contributor Author

@sergio-teruel Feel free to supersede this PR (close it and propose another PR with your additional commits). I don't plan to work this agreement stuff anymore.

@sergio-teruel
Copy link
Copy Markdown
Contributor

Ok. thanks!! I was imagining that.
So. I think that this PR must be closed

@alexis-via
Copy link
Copy Markdown
Contributor Author

Yes, you can close it when your new PR is ready (please leave a link to the new PR in the comment).

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.

10 participants