Skip to content

Conversation

@chrisandrewmann
Copy link
Contributor

Fix for issue #76

@chrisandrewmann
Copy link
Contributor Author

@etobella @pedrobaeza Could you please test when you have chance?
Noticed this intermittent issue when using spreadsheet_oca on my own system and it was annoying me, the fix seems to solve it.

Copy link
Member

@etobella etobella left a 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.

@pedrobaeza
Copy link
Member

@chrisandrewmann please follow commit message guidelines (https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#commit-message):

  • No version in the commit message (but yes on the PR title).
  • Include the module name in the commit message.

I think it's not the first time I have commented to you this.

@chrisandrewmann chrisandrewmann changed the title [18.0][FIX] Crash when editing and leaving spreadsheet fast, Component destroyed in OWL [18.0][FIX] spreadsheet_oca: Crash editing and leaving spreadsheet quickly Oct 10, 2025
…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.
@chrisandrewmann
Copy link
Contributor Author

@chrisandrewmann please follow commit message guidelines (https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#commit-message):

  • No version in the commit message (but yes on the PR title).
  • Include the module name in the commit message.

I think it's not the first time I have commented to you this.

All done, I don't remember you mentioning it before but i'll try to remember next time.
Like many of us I contribute for nothing and try to do my best to help.
Although i'm human and don't always get it right!

Copy link
Member

@pedrobaeza pedrobaeza left a 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

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-77-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 07d22bd into OCA:18.0 Oct 10, 2025
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at f31daf7. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants