Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FW][FIX] Charts: fix incoherent panel state #3576

Conversation

fw-bot
Copy link
Collaborator

@fw-bot fw-bot commented Jan 31, 2024

Description:

This task aims to address some issues in the side panel of a chart:

  1. When having only one point in a dataseries and selecting "First
    row as header", the point disappear from the chart (legit), but
    we don't have the possibility to uncheck this checkbox.
  2. When having a normal dataset and another dataset containing one
    point, checking the "... as header" checkbox works has expected, but
    unchecking it don't make the 'one-point' dataset come back

Related Task:

Task: 3380568

Related Task(s):

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 (_lt("qmsdf %s", abc))
  • unit tested
  • clean commented code
  • track breaking changes
  • doc is rebuild (npm run doc)
  • status is correct in Odoo

Forward-Port-Of: #3544
Forward-Port-Of: #2929

@robodoo
Copy link
Collaborator

robodoo commented Jan 31, 2024

@fw-bot
Copy link
Collaborator Author

fw-bot commented Jan 31, 2024

This PR targets saas-16.3 and is part of the forward-port chain. Further PRs will be created up to master.

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@fw-bot
Copy link
Collaborator Author

fw-bot commented Jan 31, 2024

@anhe-odoo @LucasLefevre ci/runbot failed on this forward-port PR

Task Description

This task aims to adress some issues in the side pannel of a chart:
1. When having only one point in a dataseries and selecting "First
row as header", the point dissapear from the chart (legit), but
we don't have the possibility to uncheck this checkbox.
2. When having a normal dataset and another dataset containing one
point, checking the "... as header" checkbox works has expected, but
unchecking it don't make the 'one-point' dataset come back

Related Task:

Task: 3380568
X-original-commit: f2a59c9
@anhe-odoo anhe-odoo force-pushed the saas-16.3-master-chart-incoherent_panel_state-anhe-n4mO-fw branch from 4c8d492 to 7ff7295 Compare January 31, 2024 13:58
@fw-bot
Copy link
Collaborator Author

fw-bot commented Jan 31, 2024

@anhe-odoo @LucasLefevre this PR was modified / updated and has become a normal PR. It should be merged the normal way (via @robodoo)

Copy link
Collaborator

@LucasLefevre LucasLefevre left a comment

Choose a reason for hiding this comment

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

robodoo r+

robodoo pushed a commit that referenced this pull request Feb 1, 2024
Task Description

This task aims to adress some issues in the side pannel of a chart:
1. When having only one point in a dataseries and selecting "First
row as header", the point dissapear from the chart (legit), but
we don't have the possibility to uncheck this checkbox.
2. When having a normal dataset and another dataset containing one
point, checking the "... as header" checkbox works has expected, but
unchecking it don't make the 'one-point' dataset come back

Related Task:

closes #3576

Task: 3380568
X-original-commit: f2a59c9
Signed-off-by: Lucas Lefèvre (lul) <lul@odoo.com>
@robodoo robodoo closed this Feb 1, 2024
@fw-bot fw-bot deleted the saas-16.3-master-chart-incoherent_panel_state-anhe-n4mO-fw branch February 15, 2024 14:47
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.

None yet

4 participants