[11.0] [MIG] contract #121
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ectly [IMP] Some conventions
…company environment Now when working on multy company environmets the server action creates invioices with accounts belong to correct company based on contract company.
pedrobaeza
force-pushed
the
11.0-mig-contract
branch
from
c58a1a5
to
b304790
Compare
|
@feketemihai I have reduced the number of commits merging "OCA Transbot..." ones following https://github.com/OCA/maintainer-tools/wiki/Merge-commits-in-pull-requests and others from same author that were split in the merge process (not squashing them on the PR). This way, we reduce a lot the size of the repo avoiding useless diffs. Finally, I have included last change that I have merged today: 7ec82c7 |
pedrobaeza
approved these changes
Nov 18, 2017
carlosdauden
approved these changes
Nov 18, 2017
|
@pedrobaeza sorry about transifex commits, i focused a lot in the Hr and Romania branch and totally forgot about this Pr... Thanks |
In order to get visibility on https://www.odoo.com/apps the OCA board has decided to add the OCA as author of all the addons maintained as part of the association.
* Tests added * Filters added. * Add _id/_ids in old property fields * Translations
pedrobaeza
force-pushed
the
11.0-mig-contract
branch
from
b304790
to
4d202de
Compare
Add template functionality for contracts
* Add a default date in the instance that an analytic account doesn’t exist (templates) - fixes OCA#68
* [IMP][9.0] contract: Add report + send by mail * Update translation and rename contract report * Changes requested and translation updated
Having a primary view that is not explicitly declared to be uses and w/o priority makes Odoo to choose between one of them randomly (well, not exactly, but kind of), so we put here which views to use. I have also put tree view as primary and put a large priority for not being selected on other actions that don't have this explicit views. A friendly name in views is also assigned.
* Use SavepointCase for making the setup only once for all tests * Make them inheritable, creating a base class with only the setup, so that it can be inherited without the need of executing all tests contained here each time you inherit it, and adding other class in the same module that inherits from the base class that actually performs the tests. * Removed duplicated test method
pedrobaeza
force-pushed
the
11.0-mig-contract
branch
from
4d202de
to
76f10fd
Compare
|
No problem. Merging. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
@pedrobaeza I made e new PR, since i had issues when fetching all commits from previous repo...