Skip to Content

Contributors

Re: Best practice add optional="show" when adding fields in tree views

I also agree with the request from a functional standpoint: especially in big repos like sale-workflow, there should be a specific attention on:

- preserving ux of runboat 

allowing to hide non-required fields in list views is particularly useful in this regard, eg. sale order lines 

- avoiding publishing modules disrupting base functional testing (eg: confirming an order, creating an invoice...)

This can be achieved by adding settings enabling the "intrusive" behavior, by not using standard records such as "Azure interior" for testing but creating a specifically named record, and in general by trying to leave runboat as it was found on a functional level :)

My 2 cents!

Francesco

Il giorno mar 29 ago 2023 alle ore 11:01 Ronald Portier <notifications@odoo-community.org> ha scritto:
Thanks Pedro, we are in complete agreement then.


Regards, Ronald


Op 29-08-2023 om 10:42 schreef Pedro M. Baeza:


> Sorry, my bad. I don't know why, I read optional="hide". It's OK to 


> put optional="show", but I would leave out of the equation mandatory 


> fields without default, or those protected with a group, as the group 


> is already the filtering method for not showing them.


>


> Regards.


>


> _______________________________________________


> Mailing-List: https://odoo-community.org/groups/contributors-15 


> <https://odoo-community.org/groups/contributors-15>


> Post to: mailto:contributors@odoo-community.org


> Unsubscribe: https://odoo-community.org/groups?unsubscribe 


> <https://odoo-community.org/groups?unsubscribe>


>

_______________________________________________
Mailing-List: https://odoo-community.org/groups/contributors-15
Post to: mailto:contributors@odoo-community.org
Unsubscribe: https://odoo-community.org/groups?unsubscribe



--

Francesco Foresti
Sicurpharma Srl
+39 333 8123 790

by Francesco Foresti - 06:21 - 14 Sep 2023

Reference

  • Best practice add optional="show" when adding fields in tree views

    Hello Contributors!

     

    I love to have your opinions here.

     

    I would recommend to PSC to review when migrating modules to ask authors for adding -> optional="show" <- to the fields they add. In order to be possible to hide.

     

    This is our sale-workflow runbot talking about sale order lines (NOTE: in v16!! In v15 is simply impossible to test anything without uninstalling 12-15 modules to have something we can call “VIEW” 😃 )

     

     

    This is the best we could have.

     

     

    Fields that cannot be hidden:

     

    • Sequence2
    • Lot_id
    • Elaborations_ids
    • Elaboration_note

     

    This best practice could apply to sale.order.lines, and all the lines in general, lis invoice lines, stock picking lines, etc…

     

    Has it any sense for you?

     

    Best regards,

    Rafael Blasco

    Moduon Team

     


    by Rafael Blasco (Moduon) - 06:50 - 28 Aug 2023