Merge lp:~yann-papouin/ocb-addons/6.1-bug-1275659-crm-my-items-filters into lp:ocb-addons/6.1

Proposed by Yann Papouin
Status: Needs review
Proposed branch: lp:~yann-papouin/ocb-addons/6.1-bug-1275659-crm-my-items-filters
Merge into: lp:ocb-addons/6.1
Diff against target: 39 lines (+4/-0)
2 files modified
crm/crm_lead_view.xml (+2/-0)
crm/crm_phonecall_view.xml (+2/-0)
To merge this branch: bzr merge lp:~yann-papouin/ocb-addons/6.1-bug-1275659-crm-my-items-filters
Reviewer Review Type Date Requested Status
Ana Juaristi Olalde (community) Approve
Stefan Rijnhart (Opener) Disapprove
Holger Brunn (Therp) Approve
Pedro Manuel Baeza Abstain
Review via email: mp+204498@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

Hi, Yann, are those filters really so needed?

review: Needs Information
Revision history for this message
Yann Papouin (yann-papouin) wrote :

It was made to mimic the 7.0/trunk behaviour where these filters are available by default.
I know that's not a major change, that's why I set the bug importance to "Wishlist".

Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

Well, for me, it's not problem to include it on OCB instead of an extra module, but let's see what others say.

Regards.

review: Abstain
Revision history for this message
Serpent Consulting Services (serpent-consulting-services) wrote :

IMHO, this is a facility, but an end user can add as many as he wants!
In this case, 'My' filters are applicable to almost every business model.

Revision history for this message
Holger Brunn (Therp) (hbrunn) wrote :

given it's in 7.0, I don't think this needs its own module

review: Approve
Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Not against this change per se, but I'm thinking that users are going to be annoyed if these untranslated buttons show up. So in this particular case I think having the improvement in a separate module in which the translations can be maintained easily should be preferred.

review: Disapprove
Revision history for this message
Ana Juaristi Olalde (ajuaristio) wrote :

I think any facility to avoid user configuring additional things is good idea.
I agree that user can create his own filter without needing a module to install, but this way he/she doesn't need to do it.

I approve since I don't see any risk.

review: Approve

Unmerged revisions

6831. By Yann Papouin

Add new filters "My Leads", "My Opportunities", "My Phonecalls"

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'crm/crm_lead_view.xml'
--- crm/crm_lead_view.xml 2013-09-09 09:32:18 +0000
+++ crm/crm_lead_view.xml 2014-02-03 14:09:39 +0000
@@ -383,6 +383,7 @@
383 <field name="name" string="Lead / Customer"383 <field name="name" string="Lead / Customer"
384 filter_domain="['|','|',('partner_name','ilike',self),('email_from','ilike',self),('name','ilike',self)]"/>384 filter_domain="['|','|',('partner_name','ilike',self),('email_from','ilike',self),('name','ilike',self)]"/>
385 <field name="user_id">385 <field name="user_id">
386 <filter domain="[('user_id','=',uid)]" help="My Leads" icon="terp-personal"/>
386 <filter icon="terp-personal-"387 <filter icon="terp-personal-"
387 domain="[('user_id','=', False)]"388 domain="[('user_id','=', False)]"
388 help="Unassigned Leads" />389 help="Unassigned Leads" />
@@ -655,6 +656,7 @@
655 <field name="name" string="Opportunity / Customer"656 <field name="name" string="Opportunity / Customer"
656 filter_domain="['|','|','|',('partner_id','ilike',self),('partner_name','ilike',self),('email_from','ilike',self),('name', 'ilike', self)]"/>657 filter_domain="['|','|','|',('partner_id','ilike',self),('partner_name','ilike',self),('email_from','ilike',self),('name', 'ilike', self)]"/>
657 <field name="user_id">658 <field name="user_id">
659 <filter domain="[('user_id','=',uid)]" help="My Opportunities" icon="terp-personal"/>
658 <filter icon="terp-personal-"660 <filter icon="terp-personal-"
659 domain="[('user_id','=', False)]"661 domain="[('user_id','=', False)]"
660 help="Unassigned Opportunities" />662 help="Unassigned Opportunities" />
661663
=== modified file 'crm/crm_phonecall_view.xml'
--- crm/crm_phonecall_view.xml 2013-09-09 09:32:18 +0000
+++ crm/crm_phonecall_view.xml 2014-02-03 14:09:39 +0000
@@ -209,6 +209,7 @@
209 <field name="name"/>209 <field name="name"/>
210 <field name="partner_id"/>210 <field name="partner_id"/>
211 <field name="user_id">211 <field name="user_id">
212 <filter domain="[('user_id','=',uid)]" help="My Phonecalls" icon="terp-personal"/>
212 <filter icon="terp-personal-"213 <filter icon="terp-personal-"
213 domain="[('user_id','=',False)]"214 domain="[('user_id','=',False)]"
214 help="Unassigned Phonecalls" />215 help="Unassigned Phonecalls" />
@@ -259,6 +260,7 @@
259 <field name="name"/>260 <field name="name"/>
260 <field name="partner_id"/>261 <field name="partner_id"/>
261 <field name="user_id">262 <field name="user_id">
263 <filter domain="[('user_id','=',uid)]" help="My Phonecalls" icon="terp-personal"/>
262 <filter icon="terp-personal-"264 <filter icon="terp-personal-"
263 domain="[('user_id','=',False)]"265 domain="[('user_id','=',False)]"
264 help="Unassigned Phonecalls" />266 help="Unassigned Phonecalls" />

Subscribers

People subscribed via source and target branches