Skip to content

[17.0][MIG] bi_view_editor_spreadsheet_dashboard#969

Merged
OCA-git-bot merged 6 commits intoOCA:17.0from
BinhexTeam:17.0-mig-bi_view_editor_spreadsheet_dashboard
Aug 19, 2025
Merged

[17.0][MIG] bi_view_editor_spreadsheet_dashboard#969
OCA-git-bot merged 6 commits intoOCA:17.0from
BinhexTeam:17.0-mig-bi_view_editor_spreadsheet_dashboard

Conversation

@antoniodavid
Copy link

@antoniodavid antoniodavid commented Jan 24, 2025

Depends on:

@antoniodavid antoniodavid mentioned this pull request Jan 24, 2025
26 tasks
@edescalona
Copy link

edescalona commented May 20, 2025

Hi @antoniodavid , could you please correct:

  1. The PR name to [17.0][MIG] bi_view_editor_spreadsheet_dashboard
  2. Add the depends on in the PR description.
  3. Change the migration commit text to that referenced in Migration to 17.0

Thank you for your contribution.

@antoniodavid antoniodavid changed the title 17.0 mig bi view editor spreadsheet dashboard [17.0][MIG] bi_view_editor_spreadsheet_dashboard May 21, 2025
Copy link

@edescalona edescalona left a comment

Choose a reason for hiding this comment

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

Hi @antoniodavid , thanks for your contribution. LGTM

One detail: The module you referenced in the description has already been merged, so the test-requirements wouldn't be necessary and the test error should be removed.

@pedrobaeza
Copy link
Member

/ocabot migration bi_view_editor_spreadsheet_dashboard

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Jun 14, 2025
@antoniodavid antoniodavid force-pushed the 17.0-mig-bi_view_editor_spreadsheet_dashboard branch from 0f92149 to 18e1f9a Compare June 14, 2025 21:18
@antoniodavid
Copy link
Author

antoniodavid commented Jun 14, 2025

/ocabot migration bi_view_editor_spreadsheet_dashboard

ping @pedrobaeza @edescalona @rrebollo

@edescalona
Copy link

Hi @antoniodavid thanks, great work.

@antoniodavid
Copy link
Author

ping @pedrobaeza

Copy link

@rrebollo rrebollo left a comment

Choose a reason for hiding this comment

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

Code Review. LGTM!

@OCA-git-bot
Copy link
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). 🤖

@HviorForgeFlow
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 579a0a5 into OCA:17.0 Aug 19, 2025
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

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.

8 participants