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

10.0 crm claim #107

Closed
wants to merge 4 commits into from
Closed

Conversation

techspawn
Copy link

crm_claim model for rma

@pedrobaeza
Copy link
Member

This still includes all the modules.

@techspawn
Copy link
Author

@pedrobaeza Please check and suggest the changes that need to be implemented

@pedrobaeza
Copy link
Member

Please squash all the current commits to only one for starting with.

help="Who is in charge of the warranty return treatment towards "
"the end customer. Company will use the current company "
"delivery or default address and so on for supplier and brand "
"manufacturer. Does not necessarily mean that the warranty "
"to be applied is the one of the return partner (ie: can be "
"returned to the company and be under the brand warranty")
warranty_return_partner = \
fields.Many2one('res.partner', string='Warranty Address',
warranty_return_partner = fields.Many2one('res.partner', string='Warranty Address',
help="Where the customer has to "
"send back the product(s)")
claim_id = fields.Many2one('crm.claim', string='Related claim',
Copy link
Member

Choose a reason for hiding this comment

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

model crm.claim does not exist anymore in Odoo 10.0 community edition

@pedrobaeza pedrobaeza mentioned this pull request Sep 11, 2017
10 tasks
@max3903 max3903 modified the milestone: 10.0 Sep 11, 2017
@bodedra
Copy link
Member

bodedra commented Oct 5, 2017

I would like to contribute here. Would you please suggest me How can I do that?

@max3903
Copy link
Member

max3903 commented Oct 24, 2017

@techspawn Please review TechspawnSolutions#1

@max3903 max3903 self-requested a review October 24, 2017 19:58
@max3903
Copy link
Member

max3903 commented Oct 24, 2017

Waiting on #126

@max3903 max3903 mentioned this pull request Oct 24, 2017
1 task
@max3903
Copy link
Member

max3903 commented Oct 24, 2017

Superseeded in #128

@pedrobaeza pedrobaeza closed this Oct 24, 2017
[FIX] PEP8 and Pylint errors
@max3903 max3903 reopened this Oct 25, 2017
@gurneyalex
Copy link
Member

gurneyalex commented Sep 25, 2018

@max3903 why did you reopen this PR ? is it still superseeded by #128 ?

@gurneyalex
Copy link
Member

Actually superseeded by OCA/crm#170

@gurneyalex gurneyalex closed this Sep 25, 2018
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.

6 participants