Skip to content

Fix to Highcharts not rendering more than 1000 datapoints#3564

Open
christian-andersen-msft wants to merge 6 commits into
mainfrom
highcharts-bug
Open

Fix to Highcharts not rendering more than 1000 datapoints#3564
christian-andersen-msft wants to merge 6 commits into
mainfrom
highcharts-bug

Conversation

@christian-andersen-msft

@christian-andersen-msft christian-andersen-msft commented Apr 24, 2025 •
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Contributor

Summary

Disables the turboThreshold option for Highcharts to improve performance

See comments to bug for detailed validation

Work Item(s)

Fixes AB#547547

@christian-andersen-msft
christian-andersen-msft requested a review from a team as a code owner April 24, 2025 14:38
@github-actions

github-actions Bot commented Apr 24, 2025

Copy link
Copy Markdown
Contributor

Issue #547547 is not valid. Please make sure you link an issue that exists, is open and is approved.

@github-actions github-actions Bot added this to the Version 27.0 milestone Apr 24, 2025
@christian-andersen-msft christian-andersen-msft changed the title Highcharts bug Fix to Highcharts not rendering more than 1000 datapoints Apr 24, 2025
msft-sam
msft-sam approved these changes Apr 28, 2025
View reviewed changes
Comment thread src/System Application/App/ControlAddIns/Resources/BusinessChart/js/BusinessChartAddIn.js
},
},
stickyTracking: false,
turboThreshold: 0,

@msft-sam msft-sam Apr 28, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice perf analysis in the bug - would be great if it is possible to log the render time + number of data points over a given threshold (e.g. 1 sec), that way we can monitor if we need to bring the threshold back or fine tune the value in the event of a livesite issue.

@aholstrup1 aholstrup1 modified the milestones: Version 27.0, Version 28.0 Aug 28, 2025
@github-actions

github-actions Bot commented Feb 8, 2026

Copy link
Copy Markdown
Contributor

⚠️ Stale Status Check Deleted

The Pull Request Build workflow run for this PR was older than 72 hours and has been deleted.

📋 Why was it deleted?

Status checks that are too old may no longer reflect the current state of the target branch. To ensure this PR is validated against the latest code and passes up-to-date checks, a fresh build is required.

🔄 How to trigger a new status check:

  1. 📤 Push a new commit to the PR branch, or
  2. 🔁 Close and reopen the PR

This will automatically trigger a new Pull Request Build workflow run.

@aholstrup1 aholstrup1 modified the milestones: Version 28.0, Version 29.0 Feb 25, 2026
@Groenbech96 Groenbech96 reopened this May 28, 2026
@Groenbech96
Groenbech96 requested a review from a team as a code owner May 28, 2026 13:58
@Groenbech96
Groenbech96 marked this pull request as draft May 28, 2026 13:59
@Groenbech96
Groenbech96 marked this pull request as ready for review May 28, 2026 13:59
@github-actions

github-actions Bot commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

⚠️ Stale Status Check Deleted

The Pull Request Build workflow run for this PR was older than 72 hours and has been deleted.

📋 Why was it deleted?

Status checks that are too old may no longer reflect the current state of the target branch. To ensure this PR is validated against the latest code and passes up-to-date checks, a fresh build is required.

🔄 How to trigger a new status check:

  1. 📤 Push a new commit to the PR branch, or
  2. 🔁 Close and reopen the PR

This will automatically trigger a new Pull Request Build workflow run.

@Groenbech96

Groenbech96 commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

@copilot pull main into this PR

Copilot AI commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

@copilot pull main into this PR

Pulled main into this PR with merge commit 90747f60.

Copilot AI requested a review from Groenbech96 June 1, 2026 05:29
@JesperSchulz JesperSchulz added Integration GitHub request for Integration area Ownership: Manual Preserve the manually selected team ownership labels Jul 17, 2026
@JesperSchulz JesperSchulz removed the Ownership: Manual Preserve the manually selected team ownership label Jul 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@darjoo darjoo darjoo approved these changes
@Kimi310 Kimi310 Kimi310 approved these changes
@msft-sam msft-sam msft-sam approved these changes
@Groenbech96 Groenbech96 Awaiting requested review from Groenbech96

At least 2 approving reviews are required to merge this pull request.

Assignees

@encimita encimita

@msft-sam msft-sam

Labels

AL: System Application Integration GitHub request for Integration area

Projects

None yet

Milestone

Version 29.0

Development

Successfully merging this pull request may close these issues.