-
-
Notifications
You must be signed in to change notification settings - Fork 66
[18.0][FIX] spreadsheet_oca: Crash editing and leaving spreadsheet quickly #77
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
@etobella @pedrobaeza Could you please test when you have chance? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For me, the changes are legit.
|
@chrisandrewmann please follow commit message guidelines (https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#commit-message):
I think it's not the first time I have commented to you this. |
…ckly When editing a spreadsheet, inserting a pivot or list or making signficant changes, exiting quickly could intermittently cause a race condition where saving does not have enough time to complete. After reloading the record, it crashes with an OWL error "Component Destroyed". This fixes the issue by awaiting the save methods before exiting.
All done, I don't remember you mentioning it before but i'll try to remember next time. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, I wasn't mean to be rude with my comment. It was just that I have seen several times this among contributors, and don't know the reasons for not putting the module name. Maybe it's stated in any place that should be corrected?
And thanks of course for the patch. We all contribute for the shared benefit.
/ocabot merge patch
|
On my way to merge this fine PR! |
|
Congratulations, your PR was merged at f31daf7. Thanks a lot for contributing to OCA. ❤️ |
Fix for issue #76