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] crm_claim_type: Migration to 12.0 #284
Conversation
…d with this, it allows to configure claim stages depending on claim types. This module includes Customer, Supplier and Other claim types as default data and its own stages relation, and also makes few stages common for different claim types
0a27741
to
4adf9ec
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for the work 👍
I did a code review: LGTM, if you like some optional changes.
</field> | ||
</record> | ||
|
||
<record id="crm_stage_claim_type_tree" model="ir.ui.view"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
optional:
use original id (crm_claim_stage_tree)
</field> | ||
</record> | ||
|
||
<record id="crm_stage_claim_type_form" model="ir.ui.view"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
optional:
use original id (crm_claim_stage_form)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested on runbot
/ocabot merge |
Rebased to 12.0-ocabot-merge-pr-284-by-pedrobaeza-bump-no, awaiting test results. |
Merged at 5447e08. Don't worry if GitHub says there are unmerged commits: it is due to a rebase before merge. All commits of this PR have been merged on the main branch. |
PR based on: #193
Rebased some commits
Cc @Tecnativa