Thursday, March 2, 2023
2 changes · master
Code cleanup and technical improvements
This change moves the main attachment capability out of a broadly used messaging component and applies it only to business records that actually need it. This should reduce unnecessary overhead across the system while preserving attachment behavior in areas such as accounting, expenses, time off, employees, and recruitment.
Original PR description
Main attachment is quite costly and as it is implemented in mail.thread and is therefore present in most model. In order to improve performance, we remove it here from mail.thread and add it through a new mixin,only on model that effectively use that features. See sub commits for more details. Task-2648976
This change narrows a costly attachment capability so it is only enabled on records that actually use it. This should improve overall system performance while preserving attachment previews and related behavior where needed.
Original PR description
Main attachment is quite costly and as it is implemented in mail.thread and is therefore present in most model. In order to improve performance, we remove it here from mail.thread and add it through a new mixin,only on model that effectively use that features. See sub commits for more details. Task-2648976