Skip to content

[9.0][MIG][website_crm_quick_answer] - #321

Merged
pedrobaeza merged 2 commits into
OCA:9.0from
Tecnativa:9.0-website_crm_quick_answer-mig
Jul 1, 2017
Merged

pedrobaeza merged 2 commits into
OCA:9.0from
Tecnativa:9.0-website_crm_quick_answer-mig

Conversation

@yajo

@yajo yajo commented Feb 16, 2017

Copy link
Copy Markdown
Member

Migration includes:

  • Drop filter and use the new domain field in the automated action.
  • Migration script to avoid FK error when dropping it.
  • Warn about non-immediate response.

This will only work after @odoo merges odoo/odoo#15528, so I'll label as WIP until then. However, code is stable.

@Tecnativa

@yajo yajo added this to the 9.0 milestone Feb 16, 2017
@yajo yajo self-assigned this Feb 16, 2017
@pedrobaeza pedrobaeza mentioned this pull request Feb 16, 2017
34 tasks
@yajo
yajo force-pushed the 9.0-website_crm_quick_answer-mig branch from 368fd57 to 9d8f5e3 Compare February 16, 2017 14:12
@yajo

yajo commented Feb 21, 2017

Copy link
Copy Markdown
Member Author

Odoo rejected odoo/odoo#15528, so it's now included in the addon, and will have to be this way until v11, so it's explained in README. Ready to review then.

@cubells cubells left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

LGTM

Code review and tested.

yajo added 2 commits May 30, 2017 10:10
Remove unexisting field.

Filter by medium too.

Allowing all users to read filter when creating a lead or an opportunity

OCA Transbot updated translations from Transifex
Migration includes:

- Drop filter and use the new domain field in the automated action.
- Migration script to avoid FK error when dropping it.
- Warn about non-immediate response.

[FIX][website_crm_quick_answer] Include odoo/odoo#15528

Patch was rejected upstream, so we will maintain it here for v9 and v10.

Fix lint error
@yajo
yajo force-pushed the 9.0-website_crm_quick_answer-mig branch from 8e68e8a to b514c4c Compare May 30, 2017 08:10
@yajo

yajo commented May 30, 2017

Copy link
Copy Markdown
Member Author

Rebased, it should go green now.

@pedrobaeza
pedrobaeza merged commit c660248 into OCA:9.0 Jul 1, 2017
@pedrobaeza
pedrobaeza deleted the 9.0-website_crm_quick_answer-mig branch July 1, 2017 19:32
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.

3 participants