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

[17.0][MIG] spreadsheet_dashboard_oca #47

Open
wants to merge 36 commits into
base: 17.0
Choose a base branch
from

Conversation

JasminSForgeFlow
Copy link

etobella and others added 30 commits September 17, 2024 12:40
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/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

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/
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
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

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/
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/
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
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
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

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/
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/
peterromao and others added 6 commits September 17, 2024 12:40
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.
@yvaucher
Copy link
Member

/ocabot migration spreadsheet_dashboard_oca

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Sep 30, 2024
@OCA-git-bot OCA-git-bot mentioned this pull request Sep 30, 2024
2 tasks
@@ -12,6 +12,7 @@ class SpreadsheetDashboard(models.Model):
_name = "spreadsheet.dashboard"
_inherit = ["spreadsheet.dashboard", "spreadsheet.abstract"]

data = fields.Binary()
Copy link
Member

Choose a reason for hiding this comment

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

Why do you need to declare the field?

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.