Skip to content

[19.0][FIX] spreadsheet_dashboard_oca: Fix the import path to correctly import the class - #135

Merged
OCA-git-bot merged 1 commit into
OCA:19.0from
Tecnativa:19.0-fix-spreadsheet_dashboard_oca-test
Oct 8, 2026
Merged

OCA-git-bot merged 1 commit into
OCA:19.0from
Tecnativa:19.0-fix-spreadsheet_dashboard_oca-test

Conversation

@carlos-lopez-tecnativa

Copy link
Copy Markdown
Contributor
image

This comes from #130.

@Tecnativa @pedrobaeza @volkantasci @etobella, could you please review this?

NOTE: This module does not run the tests automatically, but the error occurs when you try to execute the test from the UI or when another module runs the web test.

If I run the test manually, it fails. I only fixed the import, but I think the test needs a complete review. @volkantasci
image

@OCA-git-bot OCA-git-bot added mod:spreadsheet_dashboard_oca Module spreadsheet_dashboard_oca series:19.0 labels Oct 8, 2026
@pedrobaeza pedrobaeza added this to the 19.0 milestone Oct 8, 2026

@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 patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 19.0-ocabot-merge-pr-135-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit 2241730 into OCA:19.0 Oct 8, 2026
7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@carlos-lopez-tecnativa
carlos-lopez-tecnativa deleted the 19.0-fix-spreadsheet_dashboard_oca-test branch October 8, 2026 11:10
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.

4 participants