Merge lp://staging/~credativ/smile-openerp/show-all-triggers into lp://staging/smile-openerp

Proposed by Dimitri John Ledkov
Status: Merged
Approved by: Corentin Pouhet-Brunerie - http://www.Smile.fr
Approved revision: 82
Merge reported by: Corentin Pouhet-Brunerie - http://www.Smile.fr
Merged at revision: not available
Proposed branch: lp://staging/~credativ/smile-openerp/show-all-triggers
Merge into: lp://staging/smile-openerp
Diff against target: 24 lines (+4/-3)
1 file modified
smile_action_trigger/sartre_view.xml (+4/-3)
To merge this branch: bzr merge lp://staging/~credativ/smile-openerp/show-all-triggers
Reviewer Review Type Date Requested Status
Daniel Watkins (credativ) (community) Approve
Smile OpenERP Team Pending
Review via email: mp+56476@code.staging.launchpad.net

Description of the change

Change Administration/Customization/Low Level Objects/Actions/Triggers:

- removes "action" dropdown filter view
- Add "Show all" filter button
- Add "Inactive only" filter button

The default, stayed the same - display only active triggers

Now possible to view inactive triggers with a single click instead of 3 clicks.

Now possible to view *all* triggers with a single click.

To post a comment you must log in.
Revision history for this message
Daniel Watkins (credativ) (daniel-watkins-credativ) wrote :

Some odd indentation going on, otherwise if it works go ahead.

(Obviously irrelevant as it's already been merged.)

Revision history for this message
Daniel Watkins (credativ) (daniel-watkins-credativ) :
review: Approve
Revision history for this message
Dimitri John Ledkov (xnox) wrote :

Corentin Pouhet-Brunerie, are you going to merge & push this to lp:smile-openerp? I do not have push access, since I'm not part of Smile.fr.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches