Tuesday, June 29, 2021
3 changes · master
Enhancements to existing features
This update makes JavaScript test warnings easier to detect and investigate, helping developers catch problems that were previously hidden in noisy logs. It also cleans up many existing warnings across web-related tests, improving test reliability without changing end-user functionality.
Original PR description
Currently the JS logs are full of warnings and it doesn't really matter because they don't block CI. This PR: * flips JS warnings to trigger Python warnings * improves qunit's logging (hopefully) so test failures are more readable and actionable * fixes / removes / downgrades JS warnings occurring during runbot runs * also other updates probably
Event meeting rooms now receive a default Max Capacity when none is selected, preventing confusing validation errors during room creation. Chat room creation also preserves the user-selected room settings so new rooms better match organizer choices.
Original PR description
PURPOSE When one tries to create a room without selecting a max capacity it was throwing a 'Validation Error' which is not proper warning message. The purpose of this commit is to provide a default value for that field which was not getting in the respective situation, and in a way, we can eliminate this error. SPECIFICATIONS Currently, there are two solutions proposed for this task either to simply raise a proper warning message if the field is empty or override the room_max_capacity field in 'event.meeting.room' and set some default value. In this commit, we chose the second one as the solution because still we could give that default value. LINKS PR #72756 Task 2448435
The data merge screens now use clearer wording and provide helpful guidance when manual merge selections are complete. The module also removes leftover merge menu actions during uninstall, keeping menus cleaner for users after the feature is removed.
Original PR description
This commit improves merge action related naming in views. It adds action helper in depuplication view where all manual merge selection has been merged or discarded. Also, at uninstall of the data_merge module, it cleans the reminent 'Merge' server actions in contextual menu of all merge enabled models. Task ID: 2559013