Monday, October 30, 2017
3 changes · master
Resolved issues and error corrections
Fixes kanban navigation so that after users load more cards in a column, opening a card and using next/previous in the form view follows the full loaded list. This prevents users from being skipped to another stage before seeing the remaining tasks in the current stage.
Original PR description
…rm view navigate next record skip newly loaded record Description of the issue/feature this PR addresses: When I'm on a kanban view, I select the last visible task (there's some remaining tasks that…
…rm view navigate next record skip newly loaded record Description of the issue/feature this PR addresses: When I'm on a kanban view, I select the last visible task (there's some remaining tasks that I can load) of the column and open it. now in form view click on navigation arrow.It does not show loaded records. Instead of showing me the next task in the same column (so the "remaining tasks"), It jumps to the other column (stage ). Current behavior before PR: When I'm on a kanban view, I select the last visible task (there's some remaining tasks that I can load) of the column and open it. now in form view click on navigation arrow.It does not show loaded records. Instead of showing me the next task in the same column (so the "remaining tasks"), It jumps to the other column (stage ). Desired behavior after PR is merged: When load more is clicked, then in the form view of task it will show all the loaded records and then it will jump to the next column(stage). -- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Task: https://www.odoo.com/web#id=34038&view_type=form&model=project.task&action=327&menu_id=4720 Pad: https://pad.odoo.com/p/r.bd9c509ceba6177571e51c8a9844d069
This update corrects spelling and grammar mistakes in developer-facing comments and messages across Odoo's core tooling. It improves clarity and professionalism without changing business features or system behavior.
Original PR description
Description of the issue/feature this PR addresses: Basic spelling and grammar corrections. Current behavior before PR: Comments and output with spelling errors Desired behavior after PR is merged: Correct majority of spelling and grammar issues. Ignoring large amount of software development and international spelling issues. -- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
Code cleanup and technical improvements
This change updates the app switcher code in Enterprise and Studio to follow newer JavaScript coding guidelines. It is an internal cleanup intended to make the code easier to maintain, with no expected change to day-to-day user behavior.
Original PR description
see https://github.com/odoo/odoo/wiki/Javascript-coding-guidelines.
Files changed:
- _web_enterprise/static/src/js/app_switcher.js_
- _web_studio/static/src/js/app_switcher.js_
Some exceptions to the new coding guidelines:
- _on_attach_callback_ and _on_detach_callback_
- fields of objects that are used by many modules
(e.g. 'MenuItem' with 'menu_id')
Note:
- this commit relies on a newly added function in community:
_/odoo/addons/web/static/src/js/utility.js_
new function: **traversePath**
- this function is exactly similar to the old 'visit' in
_web_enterprise/static/src/js/app_switcher.js_
see https://github.com/odoo-dev/odoo/commit/b6dfd5726e8b68024c8448a61d788cc586b70a54