Skip to content

[19.0][MIG] spreadsheet_dashboard_purchase_stock_oca: Migration to version 19.0 - #120

Merged
OCA-git-bot merged 10 commits into
OCA:19.0from
Tecnativa:19.0-mig-spreadsheet_dashboard_purchase_stock_oca
Aug 14, 2026
Merged

[19.0][MIG] spreadsheet_dashboard_purchase_stock_oca: Migration to version 19.0#120
OCA-git-bot merged 10 commits into
OCA:19.0from
Tecnativa:19.0-mig-spreadsheet_dashboard_purchase_stock_oca

Conversation

@pilarvargas-tecnativa

Copy link
Copy Markdown

@OCA-git-bot OCA-git-bot added mod:spreadsheet_dashboard_purchase_stock_oca Module spreadsheet_dashboard_purchase_stock_oca series:19.0 labels Jul 21, 2026
@pilarvargas-tecnativa
pilarvargas-tecnativa force-pushed the 19.0-mig-spreadsheet_dashboard_purchase_stock_oca branch from 8c60703 to 60c36f9 Compare July 21, 2026 12:57
@pilarvargas-tecnativa
pilarvargas-tecnativa marked this pull request as draft July 22, 2026 06:20
@pilarvargas-tecnativa
pilarvargas-tecnativa marked this pull request as ready for review July 22, 2026 06:27

@victoralmau victoralmau 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.

Code review OK

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@pilarvargas-tecnativa
pilarvargas-tecnativa marked this pull request as draft July 27, 2026 13:56
@pilarvargas-tecnativa
pilarvargas-tecnativa marked this pull request as ready for review August 5, 2026 06:49
@pilarvargas-tecnativa

Copy link
Copy Markdown
Author

ping @pedrobaeza

@pedrobaeza

Copy link
Copy Markdown
Member

I don't think you should depend on spreadsheet_dashboard_purchase_oca, but just add a new entry in the dashboards with just the specific things related to stock.

@pilarvargas-tecnativa
pilarvargas-tecnativa force-pushed the 19.0-mig-spreadsheet_dashboard_purchase_stock_oca branch from 0a304d5 to c6cdf64 Compare August 11, 2026 08:01
@pilarvargas-tecnativa

Copy link
Copy Markdown
Author

I don't think you should depend on spreadsheet_dashboard_purchase_oca, but just add a new entry in the dashboards with just the specific things related to stock.

done

image

@pedrobaeza

Copy link
Copy Markdown
Member

The only stock related thing I see is the last one. The rest are common to purchase, not to stock.

@pilarvargas-tecnativa
pilarvargas-tecnativa force-pushed the 19.0-mig-spreadsheet_dashboard_purchase_stock_oca branch from c6cdf64 to dc4f0a0 Compare August 11, 2026 12:57
@pilarvargas-tecnativa

Copy link
Copy Markdown
Author

The only stock related thing I see is the last one. The rest are common to purchase, not to stock.

Done

@pedrobaeza

Copy link
Copy Markdown
Member

/ocabot migration spreadsheet_dashboard_purchase_stock_oca

On enterprise, I see these valuable metrics related to stock:

  • KPI of "Supplier service level".
  • KPI of "On time deliveries".
  • Bar graph of "% On time deliveries by vendor"

Can you please add them?

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone Aug 11, 2026
@OCA-git-bot OCA-git-bot mentioned this pull request Aug 11, 2026
4 tasks
@pilarvargas-tecnativa

Copy link
Copy Markdown
Author

@pedrobaeza please review

@pedrobaeza

Copy link
Copy Markdown
Member

Red CI

@pilarvargas-tecnativa
pilarvargas-tecnativa force-pushed the 19.0-mig-spreadsheet_dashboard_purchase_stock_oca branch from 5a58ffd to f66ffa0 Compare August 12, 2026 10:54
@pedrobaeza

Copy link
Copy Markdown
Member

The on time metrics seem to not work correctly:

imagen imagen

@pilarvargas-tecnativa

Copy link
Copy Markdown
Author

The on time metrics seem to not work correctly:

Perhaps the dashboard hadn’t been refreshed? I’ve logged into Runboat to check, and the following data is displayed:

http://oca-spreadsheet-19-0-pr120-f66ffa0da183.runboat.odoo-community.org/odoo/dashboards?dashboard_id=4

image

@pedrobaeza

Copy link
Copy Markdown
Member

OK, it seems to be a glitch.

Can you do another thing? Please put this one and the other into a new section "Purchases" instead of "Logistics", which is not the right one. You will need to create it in the other module, and depend this one on the other.

Odoo and others added 9 commits August 13, 2026 15:52
…rsion

Copied as is from Odoo 17.0 branch. Later commits will do the needed
changes.
Currently translated at 100.0% (41 of 41 strings)

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

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

Translation: spreadsheet-18.0/spreadsheet-18.0-spreadsheet_dashboard_purchase_stock_oca
Translate-URL: https://translation.odoo-community.org/projects/spreadsheet-18-0/spreadsheet-18-0-spreadsheet_dashboard_purchase_stock_oca/fr/
@pilarvargas-tecnativa
pilarvargas-tecnativa force-pushed the 19.0-mig-spreadsheet_dashboard_purchase_stock_oca branch 2 times, most recently from 9dc9db1 to 7feb2f5 Compare August 13, 2026 14:30
@pedrobaeza

Copy link
Copy Markdown
Member

"Purchase receipts" entry should go after "Vendors" one.

@pilarvargas-tecnativa
pilarvargas-tecnativa force-pushed the 19.0-mig-spreadsheet_dashboard_purchase_stock_oca branch from 7feb2f5 to 4758d8a Compare August 14, 2026 05:40
@pilarvargas-tecnativa

Copy link
Copy Markdown
Author

"Purchase receipts" entry should go after "Vendors" one.

Done

image

@pedrobaeza

Copy link
Copy Markdown
Member

/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 19.0-ocabot-merge-pr-120-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit 4ab2a88 into OCA:19.0 Aug 14, 2026
7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@pedrobaeza
pedrobaeza deleted the 19.0-mig-spreadsheet_dashboard_purchase_stock_oca branch August 14, 2026 06:34
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.

9 participants