Skip to content

[19.0][MIG] spreadsheet_dashboard_oca: Migration to 19.0 - #123

Merged
OCA-git-bot merged 27 commits into
OCA:19.0from
chrisandrewmann:19.0-mig-spreadsheet_dashboard_oca
Jul 28, 2026
Merged

[19.0][MIG] spreadsheet_dashboard_oca: Migration to 19.0#123
OCA-git-bot merged 27 commits into
OCA:19.0from
chrisandrewmann:19.0-mig-spreadsheet_dashboard_oca

Conversation

@chrisandrewmann

@chrisandrewmann chrisandrewmann commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Migration of spreadsheet_dashboard_oca to 19.0

Nothing much appeared to be needed apart from:

  • self.env._ to replace imported translations
    The two failing checks are Codecov (patch/project coverage), because module has no test suite.

Tested manually and works locally.

etobella and others added 26 commits July 28, 2026 10:32
Currently translated at 80.0% (4 of 5 strings)

Translation: spreadsheet-16.0/spreadsheet-16.0-spreadsheet_dashboard_oca
Translate-URL: https://translation.odoo-community.org/projects/spreadsheet-16-0/spreadsheet-16-0-spreadsheet_dashboard_oca/it/
Currently translated at 100.0% (9 of 9 strings)

Translation: spreadsheet-16.0/spreadsheet-16.0-spreadsheet_dashboard_oca
Translate-URL: https://translation.odoo-community.org/projects/spreadsheet-16-0/spreadsheet-16-0-spreadsheet_dashboard_oca/fr_FR/
Currently translated at 77.7% (7 of 9 strings)

Translation: spreadsheet-16.0/spreadsheet-16.0-spreadsheet_dashboard_oca
Translate-URL: https://translation.odoo-community.org/projects/spreadsheet-16-0/spreadsheet-16-0-spreadsheet_dashboard_oca/nl/
Currently translated at 100.0% (9 of 9 strings)

Translation: spreadsheet-16.0/spreadsheet-16.0-spreadsheet_dashboard_oca
Translate-URL: https://translation.odoo-community.org/projects/spreadsheet-16-0/spreadsheet-16-0-spreadsheet_dashboard_oca/it/
All the spreadsheet dashboards coming from Odoo in the modules
spreadsheet_dashboard* are noupdate="0", which means that if you
modify anything via this module, and then you update the module, all the
changes will be lost.

To avoid frustrations and to allow seamless updates, the following
mechanisms have been put in place:

- Spreadsheet dashboards now have an active field.
- Only the manually created dashboards or those coming from data
  with noupdate="1" will be editable.
- There's a mechanism for copying existing dashboards.

So, for modifying one of the standard dashboards, the steps will be:

- Duplicate it through the "Copy" button.
- Disable the standard one.
- Edit the copy.

TT49379
Currently translated at 100.0% (12 of 12 strings)

Translation: spreadsheet-16.0/spreadsheet-16.0-spreadsheet_dashboard_oca
Translate-URL: https://translation.odoo-community.org/projects/spreadsheet-16-0/spreadsheet-16-0-spreadsheet_dashboard_oca/it/
…board_sheet + dynamic + new dashboard

- Add field to define name of table and restructure order of fields

- Error with dashboard_sheet selected

The error is happening because the dashboard where
the data has to be added is not being showed.

With this changes the field to define the
dashboard is required and is showed.

Added a way to limit the search to the dashboards
that can be edited

- Allow to generate dynamic tables

On dashboards exists tables that are updated dynamically,
with this changes we have the option to generate this
dynamic tables using the pivot inserter.

Added a button to duplicate and change to dynamic
table the selected table on spreadsheet editor.

If a table with multiple row goups generated, the
way to generate the dynamic table will be set the
number of rows for each level of indentation.

Example:
2 rowGroupBys
3 number of rows
- val1
    - subval1.1
    - subval1.2
    - subval1.3
- val2
    - subval2.1
    - subval2.2
    - subval2.3
- val3
    - subval3.1
    - subval3.2
    - subval3.3

- Add option to create new dashboard
Currently translated at 100.0% (16 of 16 strings)

Translation: spreadsheet-16.0/spreadsheet-16.0-spreadsheet_dashboard_oca
Translate-URL: https://translation.odoo-community.org/projects/spreadsheet-16-0/spreadsheet-16-0-spreadsheet_dashboard_oca/it/
Currently translated at 100.0% (16 of 16 strings)

Translation: spreadsheet-16.0/spreadsheet-16.0-spreadsheet_dashboard_oca
Translate-URL: https://translation.odoo-community.org/projects/spreadsheet-16-0/spreadsheet-16-0-spreadsheet_dashboard_oca/pt/
With these changes, we have added the ability to make the columns
dynamic, just as is done with the rows.

To add this type of table, we will need to select the dynamic columns
option and choose the number of columns to insert. Keep in mind that if
multiple column levels are selected, the chosen number will apply to
each of these levels.

In addition to this, the wizard that opens with the spreadsheet button
has been updated to allow us to choose the type of dynamism we expect in
the inserted table.
- Rename field data to spreadsheet_binary_data in line with v17 spreadsheet_oca changes in PR (#60)
- Remove dependency on base_view_inheritance_extension module.
- Remove dependency on spreadsheet.mixin, moved to spreadsheet.abstract
Currently translated at 100.0% (17 of 17 strings)

Translation: spreadsheet-17.0/spreadsheet-17.0-spreadsheet_dashboard_oca
Translate-URL: https://translation.odoo-community.org/projects/spreadsheet-17-0/spreadsheet-17-0-spreadsheet_dashboard_oca/zh_CN/
…ertions

1- Removed the option to add in a new sheet; this will now be the default behavior of the option to add to an existing spreadsheet. Due to the nature of tables, they are always dynamic and structured in such a way that cells don’t contain static content, making it impossible to determine where previously inserted tables end within the spreadsheet. Therefore, we’ve chosen to follow Odoo’s approach, which always inserts data into new sheets.

2- Removed the use of dynamic column fields, as they are no longer used. Only dynamic rows are used now, for cases when a tree view is inserted.
Currently translated at 100.0% (16 of 16 strings)

Translation: spreadsheet-18.0/spreadsheet-18.0-spreadsheet_dashboard_oca
Translate-URL: https://translation.odoo-community.org/projects/spreadsheet-18-0/spreadsheet-18-0-spreadsheet_dashboard_oca/it/
Currently translated at 100.0% (30 of 30 strings)

Translation: spreadsheet-18.0/spreadsheet-18.0-spreadsheet_dashboard_oca
Translate-URL: https://translation.odoo-community.org/projects/spreadsheet-18-0/spreadsheet-18-0-spreadsheet_dashboard_oca/it/
Currently translated at 100.0% (30 of 30 strings)

Translation: spreadsheet-18.0/spreadsheet-18.0-spreadsheet_dashboard_oca
Translate-URL: https://translation.odoo-community.org/projects/spreadsheet-18-0/spreadsheet-18-0-spreadsheet_dashboard_oca/es/
Currently translated at 50.0% (15 of 30 strings)

Translation: spreadsheet-18.0/spreadsheet-18.0-spreadsheet_dashboard_oca
Translate-URL: https://translation.odoo-community.org/projects/spreadsheet-18-0/spreadsheet-18-0-spreadsheet_dashboard_oca/fr/
@OCA-git-bot OCA-git-bot added mod:spreadsheet_dashboard_oca Module spreadsheet_dashboard_oca series:19.0 labels Jul 28, 2026
@pedrobaeza

Copy link
Copy Markdown
Member

/ocabot migration spreadsheet_dashboard_oca

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone Jul 28, 2026

@carlos-lopez-tecnativa carlos-lopez-tecnativa left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just a minor improvement: please use the new Command syntax instead of the old syntax.
TT58718

Comment thread spreadsheet_dashboard_oca/data/spreadsheet_spreadsheet_import_mode.xml Outdated
Comment thread spreadsheet_dashboard_oca/data/spreadsheet_spreadsheet_import_mode.xml Outdated
@chrisandrewmann
chrisandrewmann force-pushed the 19.0-mig-spreadsheet_dashboard_oca branch from cc9c81e to f809cae Compare July 28, 2026 15:29
@chrisandrewmann

Copy link
Copy Markdown
Contributor Author

LGTM, just a minor improvement: please use the new Command syntax instead of the old syntax.

@carlos-lopez-tecnativa thanks, all done

@pedrobaeza pedrobaeza left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 19.0-ocabot-merge-pr-123-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit e10e79a into OCA:19.0 Jul 28, 2026
5 of 7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@chrisandrewmann
chrisandrewmann deleted the 19.0-mig-spreadsheet_dashboard_oca branch July 28, 2026 16:44
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.