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

Add module crm_lead_website. #53

Merged
merged 6 commits into from
Oct 30, 2015
Merged

Add module crm_lead_website. #53

merged 6 commits into from
Oct 30, 2015

Conversation

yajo
Copy link
Member

@yajo yajo commented Oct 6, 2015

Relaunching part of Antiun/antiun-odoo-addons#11.

Add website field to leads.

It will fully work after odoo/odoo#8899 is merged, but I think it can be merged anyway.

cc @rafaelbn.

@yajo
Copy link
Member Author

yajo commented Oct 13, 2015

Now NOT depending on odoo/odoo#8899.

@rafaelbn
Copy link
Member

Tested in runbot 👍

@pedrobaeza
Copy link
Member

Please check Travis

@yajo
Copy link
Member Author

yajo commented Oct 20, 2015

Travis seems unrelated. Same as #52 (comment).

@janetrp
Copy link

janetrp commented Oct 22, 2015

website field appears in lead form. 👍

@yajo
Copy link
Member Author

yajo commented Oct 26, 2015

Travis is fixed, please check again. 😄

@carlosdauden
Copy link

👍

if partner_id:
partner = self.pool['res.partner'].browse(cr, uid, partner_id,
context=context)
partner = self.env['res.partner'].browse(partner_id)
res['value'].update({'street3': partner.street3})
return res

Choose a reason for hiding this comment

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

This file belongs to this module?
Is merged in #61.

Copy link
Member Author

Choose a reason for hiding this comment

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

True, I added it here to get green marks too, but I'll merge now.

@rafaelbn
Copy link
Member

@yajo there is not website in the opportunity. You must add also in opportunity. Check screenshoot
29-10-2015 16-39-19

@yajo
Copy link
Member Author

yajo commented Oct 29, 2015

Good points, thanks.

@rafaelbn rafaelbn mentioned this pull request Oct 29, 2015
@yajo
Copy link
Member Author

yajo commented Oct 29, 2015

All should be fixed.

@carlosdauden
Copy link

👍

1 similar comment
@rafaelbn
Copy link
Member

👍

rafaelbn added a commit that referenced this pull request Oct 30, 2015
Add module crm_lead_website.
@rafaelbn rafaelbn merged commit 62bda37 into OCA:8.0 Oct 30, 2015
@yajo yajo deleted the lead-website branch November 23, 2015 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants