Code review comment for lp:~therp-nl/account-financial-tools/7.0-add_move_line_no_default_search_period_journal

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

Hi, Stefan,

I have tested the module and now I see no reason for the new menu I proposed, because it only changes default values. It doesn't remove combo-boxes, which is what I understand at the beginning.

It works perfectly.

Thanks for the contribution.

Regards.

review: Approve (code review and test)

« Back to merge proposal