Skip to content
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

[16.0] Add module spreadsheet_dashboard_force_db_storage #440

Conversation

grindtildeath
Copy link
Member

Spreadsheet Dashboard Force DB storage

This module allows to force storage of spreadsheet dashboard data in the database
instead of any external storage that is configured.

When using base_attachment_object_storage, Odoo attachments' binaries data that
are stored on object storage use a storage key that is computed according to the
content of the binary data (ie checksum).
It works well as long as they are not meant to be modified, since it avoids duplication
of identical content on the object storage.
However, the spreadsheet dashboard records' binary data is meant to be modified and
the default worksheet data still has to be duplicated among the spreadsheet dashboard
records, to allow modification for each spreadsheet.

@grindtildeath grindtildeath force-pushed the 16.0-add-spreadsheet_dashboard_force_db_storage branch from 37b342e to e44a529 Compare October 11, 2023 11:48
@codecov-commenter
Copy link

codecov-commenter commented Oct 11, 2023

Codecov Report

Merging #440 (dea6ab2) into 16.0 (0d0ca60) will increase coverage by 1.17%.
The diff coverage is 70.37%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@            Coverage Diff             @@
##             16.0     #440      +/-   ##
==========================================
+ Coverage   36.05%   37.22%   +1.17%     
==========================================
  Files          33       37       +4     
  Lines         760      787      +27     
  Branches      106      110       +4     
==========================================
+ Hits          274      293      +19     
- Misses        480      487       +7     
- Partials        6        7       +1     
Files Coverage Δ
spreadsheet_dashboard_force_db_storage/__init__.py 100.00% <100.00%> (ø)
...heet_dashboard_force_db_storage/models/__init__.py 100.00% <100.00%> (ø)
...d_force_db_storage/models/spreadsheet_dashboard.py 100.00% <100.00%> (ø)
spreadsheet_dashboard_force_db_storage/hooks.py 55.55% <55.55%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@grindtildeath grindtildeath force-pushed the 16.0-add-spreadsheet_dashboard_force_db_storage branch from e44a529 to dea6ab2 Compare October 11, 2023 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants