openerp-community-reviewer team mailing list archive

[Leonardo Pistone - camptocamp <leonardo.pistone@xxxxxxxxxxxxxx>]

Leonardo Pistone - camptocamp has proposed merging lp:~camptocamp/account-financial-tools/7.0-batch-validate-replace-menuitem-lep into lp:account-financial-tools.

Requested reviews:
  Account Core Editors (account-core-editors)

For more details, see:
https://code.launchpad.net/~camptocamp/account-financial-tools/7.0-batch-validate-replace-menuitem-lep/+merge/208412

account_move_batch_validate: the old, asynchronous menuitem is now a bit useless and confusing. we then replace it
-- 
https://code.launchpad.net/~camptocamp/account-financial-tools/7.0-batch-validate-replace-menuitem-lep/+merge/208412
Your team Account Core Editors is requested to review the proposed merge of lp:~camptocamp/account-financial-tools/7.0-batch-validate-replace-menuitem-lep into lp:account-financial-tools.

=== modified file 'account_move_batch_validate/wizard/move_marker_view.xml'
--- account_move_batch_validate/wizard/move_marker_view.xml	2014-02-21 13:44:39 +0000
+++ account_move_batch_validate/wizard/move_marker_view.xml	2014-02-26 15:47:43 +0000
@@ -50,16 +50,10 @@
             <field name="target">new</field>
         </record>
 
-        <menuitem
-            name="Mark Journal Items for Batch Posting"
-            parent="account.periodical_processing_journal_entries_validation"
-            action="action_account_move_marker"
-            id="menu_account_move_marker"
-            icon="STOCK_PRINT"/>
-
-        <!-- hide the existing menuitem -->
+        <!-- replace existing menuitem -->
         <record id="account.menu_validate_account_moves" model="ir.ui.menu">
-            <field name="groups_id" eval="[(6, 0, [ref('base.group_no_one')])]"/>
+            <field name="name">Mark Journal Items for Batch Posting</field>
+            <field name="action" ref="action_account_move_marker" />
         </record>
 
     </data>