Skip to content

Updates for tests #46

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

Merged
merged 7 commits into from
Aug 11, 2025
Merged

Updates for tests #46

merged 7 commits into from
Aug 11, 2025

Conversation

marviem
Copy link
Contributor

@marviem marviem commented Aug 4, 2025

list of updates to react projects:

  • updated packages
  • updated chart demos to match the vue projects
  • migrated highcharts to the official highcharts-react wrapper
  • fixed a few bugs with amcharts
  • fixed a bug with printing logs in the next.js events demo
  • other minor updates

@marviem marviem requested review from VeraDidenko and Copilot August 4, 2025 19:04
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates React projects to modernize dependencies, standardize chart demos across platforms, and migrate Highcharts integration to use the official wrapper. The key changes focus on package updates, chart demo improvements, and bug fixes.

  • Updated packages to latest versions including React 19, TypeScript 5.8, and chart libraries
  • Migrated Highcharts integration from direct DOM manipulation to official React wrapper
  • Standardized chart demos and reports across Vue and React projects for consistency

Reviewed Changes

Copilot reviewed 36 out of 36 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
typescript/package.json, nextjs-ts/package.json, ES6/package.json Updated dependencies to latest versions
WithHighcharts.tsx/.jsx Migrated to highcharts-react-official wrapper with state management
WithAmcharts4.tsx/.jsx, WithAmcharts.tsx/.jsx Improved chart lifecycle management and consistent pie chart implementation
UsingAPICalls.tsx/.jsx Renamed functions for better clarity and removed unused code
HandlingEvents.tsx/.jsx Fixed event handling and improved state management
Various component files Standardized report URLs and improved code formatting

@marviem marviem merged commit d3ad0a7 into master Aug 11, 2025
0 of 2 checks passed
@marviem marviem deleted the updates-for-tests branch August 11, 2025 11:57
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.

2 participants