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

[14.0][backport] project_forecast_line_bokeh_chart #1254

Conversation

ntsirintanis
Copy link
Contributor

project_forecast_line_bokeh_chart from 15.0 to 14.0

yankinmax and others added 8 commits January 18, 2024 11:22
Enhance the display in the bokeh chart to better display negative
consolidated capacity.

Without the fix, the stacked vbar would display the negative capacity
bar starting from the top of the stack and going down, superposed on
positive capacity bars, which was confusing.

This fix gathers overload capacity (negative consolidated capacities)
and plots them in a separacte vbar with only negative values, which
produces an easier to understand chart.

See bokeh/bokeh#9011 for a discussion of this
stacked bar chart behavior in bokeh.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: project-15.0/project-15.0-project_forecast_line_bokeh_chart
Translate-URL: https://translation.odoo-community.org/projects/project-15-0/project-15-0-project_forecast_line_bokeh_chart/
@dreispt
Copy link
Sponsor Member

dreispt commented Apr 30, 2024

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 14.0-ocabot-merge-pr-1254-by-dreispt-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Apr 30, 2024
Signed-off-by dreispt
@OCA-git-bot
Copy link
Contributor

@dreispt your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-1254-by-dreispt-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

1 similar comment
@OCA-git-bot
Copy link
Contributor

@dreispt your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-1254-by-dreispt-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

@dreispt
Copy link
Sponsor Member

dreispt commented May 18, 2024

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 8d401da into OCA:14.0 May 18, 2024
7 of 9 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at f28c023. 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.

None yet

7 participants