Skip to content

[FIX] pivot cell style copied twice#8255

Closed
matho-odoo wants to merge 1 commit intosaas-19.2from
saas-19.2-fix-pivot-style-matho
Closed

[FIX] pivot cell style copied twice#8255
matho-odoo wants to merge 1 commit intosaas-19.2from
saas-19.2-fix-pivot-style-matho

Conversation

@matho-odoo
Copy link
Copy Markdown
Contributor

If we copy the top left cell or the all pivot table, we should not copy the style of the cell

Task: 5909138

review checklist

  • feature is organized in plugin, or UI components
  • support of duplicate sheet (deep copy)
  • in model/core: ranges are Range object, and can be adapted (adaptRanges)
  • in model/UI: ranges are strings (to show the user)
  • undo-able commands (uses this.history.update)
  • multiuser-able commands (has inverse commands and transformations where needed)
  • new/updated/removed commands are documented
  • exportable in excel
  • translations (_t("qmsdf %s", abc))
  • unit tested
  • clean commented code
  • track breaking changes
  • doc is rebuild (npm run doc)
  • status is correct in Odoo

If we copy the top left cell or the all pivot table,
we should not copy the style of the cell

Task: 5909138
@robodoo
Copy link
Copy Markdown
Collaborator

robodoo commented Mar 31, 2026

Pull request status dashboard

Copy link
Copy Markdown
Contributor

@hokolomopo hokolomopo left a comment

Choose a reason for hiding this comment

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

robodoo pushed a commit that referenced this pull request Apr 13, 2026
If we copy the top left cell or the all pivot table,
we should not copy the style of the cell

closes #8255

Task: 5909138
Signed-off-by: Adrien Minne (adrm) <adrm@odoo.com>
@robodoo robodoo closed this Apr 13, 2026
@fw-bot fw-bot mentioned this pull request Apr 13, 2026
14 tasks
@fw-bot fw-bot deleted the saas-19.2-fix-pivot-style-matho branch April 20, 2026 07:36
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.

3 participants