Repository navigation
[20.0][MIG] spreadsheet_dashboard_purchase_oca: Migration to 20.0 - #133
Conversation
Copied as is from Odoo 17.0 branch. Later commits will do the needed changes.
Currently translated at 100.0% (27 of 27 strings) Translation: spreadsheet-18.0/spreadsheet-18.0-spreadsheet_dashboard_purchase_oca Translate-URL: https://translation.odoo-community.org/projects/spreadsheet-18-0/spreadsheet-18-0-spreadsheet_dashboard_purchase_oca/it/
Currently translated at 100.0% (27 of 27 strings) Translation: spreadsheet-18.0/spreadsheet-18.0-spreadsheet_dashboard_purchase_oca Translate-URL: https://translation.odoo-community.org/projects/spreadsheet-18-0/spreadsheet-18-0-spreadsheet_dashboard_purchase_oca/fr/
Currently translated at 100.0% (31 of 31 strings) Translation: spreadsheet-19.0/spreadsheet-19.0-spreadsheet_dashboard_purchase_oca Translate-URL: https://translation.odoo-community.org/projects/spreadsheet-19-0/spreadsheet-19-0-spreadsheet_dashboard_purchase_oca/it/
Signed-off-by: Don Kendall <dkendall@ledoweb.com> Assisted-by: Claude Code
OCA-git-bot
added
mod:spreadsheet_dashboard_purchase_oca
|
The dashboard styles need some adjustment for 20.0. At least the background appears purple and looks inconsistent with the standard dashboards. Could you review the dashboard styling and align it with the current design? @Tecnativa TT64884 |
8d44ea1 to
166c1f9
Compare
|
@pilarvargas-tecnativa updated. I missed it because my production instance overrides that. so it showed up as my own brand amber. |
|
@carlos-lopez-tecnativa please review |
|
Have you checked in runbot how the enterprise dashboard looks on 20? For getting/mimicking ideas from it. |
The KPI cards at the top now have white backgrounds, and each section, such as “Top Products”, has a white card wrapping both its title and content. In the migrated dashboard (left screenshot), the “Top Vendors” background only covers part of the section, while “Top Orders” and the other sections lack that white background. These are the styling changes that need to be applied in this migration. |
|
I had looked as well and already done that. will push here now. |
166c1f9 to
3455f70
Compare
Assisted-by: Claude Opus 5
|
Something has thrown off the sizing of the top KPI cards 
And the layout of the other tables is also different. 
|
* data/dashboards.xml: payload loads with type="bytes", not type="base64". * vendors_dashboard.json: Dashboard sheet backgroundColor #F9FAFB, as core. * vendors_dashboard.json: adopt the 20.0 dashboard styling — white KPI cards and a card around each section's title and content, on the original layout. Signed-off-by: Don Kendall <dkendall@ledoweb.com> Assisted-by: Claude Opus 5
3455f70 to
b93205a
Compare
pedrobaeza
left a comment
pedrobaeza
left a comment
There was a problem hiding this comment.
Tested on runboat
/ocabot merge patch
|
This PR looks fantastic, let's merge it! |
|
Congratulations, your PR was merged at 6d1d7bb. Thanks a lot for contributing to OCA. ❤️ |
Port of
spreadsheet_dashboard_purchase_ocafrom19.0to20.0. Follows the OCA migration guide.Non-mechanical adaptations worth flagging
data/files/vendors_dashboard.json). 19.0 forced the dashboard grid white fromspreadsheet_dashboard/.../dashboard_action.scss(.o_spreadsheet_dashboard_action { background-color: white; … .o-grid { background-color: white } }). 20.0 dropped both rules and instead sets--os-canvas-background-color: #{$o-navbar-background}, while o-spreadsheet gained a per-sheetbackgroundColorwhose renderer falls back toctx.clearRect()when it is unset — so a sheet carrying nobackgroundColornow paints transparent over the navbar colour. All 28 core 20.0 dashboard payloads were regenerated with"backgroundColor": "#F9FAFB"; this port sets the same value on theDashboardsheet. The rest of the payload is unchanged from 19.0.AI-assisted (Claude Code); every change reviewed, tested, and owned by the author.