Skip to content

[8.19] Upgrade @elastic/charts to 69.2.1 (#216430)#217134

Merged
markov00 merged 16 commits intoelastic:8.19from
markov00:backport/8.x/pr-216430
May 9, 2025
Merged

[8.19] Upgrade @elastic/charts to 69.2.1 (#216430)#217134
markov00 merged 16 commits intoelastic:8.19from
markov00:backport/8.x/pr-216430

Conversation

@markov00
Copy link
Contributor

@markov00 markov00 commented Apr 4, 2025

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

## Summary

This PR updates `@elastic/charts` library to 69.2.1, that reverse the
wrongly merged multi-time axis breaking change and keeps only the new
feature added to Metric about exposing the fontsize and the color
styling

[v69.2.0](https://github.com/elastic/elastic-charts/blob/HEAD/CHANGELOG.md#6920-2025-03-26)
[Compare
Source](https://github.com/elastic/elastic-charts/compare/v69.1.1...v69.2.0)

Features
metric: Expose both font size and color stylings on the extra prop
([#​2627](https://github.com/elastic/elastic-charts/issues/2627))
([2d21118](https://github.com/elastic/elastic-charts/commit/2d2111899588d6619dbe4152fd8113cd2215fa1a))

(cherry picked from commit f8b3e93)

# Conflicts:
#	package.json
#	yarn.lock
@markov00 markov00 requested a review from kibanamachine as a code owner April 4, 2025 07:34
@markov00 markov00 added the backport This PR is a backport of another PR label Apr 4, 2025
@markov00 markov00 enabled auto-merge (squash) April 4, 2025 07:34
@markov00 markov00 marked this pull request as draft April 7, 2025 10:27
auto-merge was automatically disabled April 7, 2025 10:27

Pull request was converted to draft

Copy link
Contributor

@nickofthyme nickofthyme left a comment

Choose a reason for hiding this comment

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

Changes LGTM, tested and see no issues.

@markov00 markov00 marked this pull request as ready for review April 8, 2025 10:18
@markov00 markov00 marked this pull request as draft April 8, 2025 10:18
@markov00
Copy link
Contributor Author

markov00 commented Apr 8, 2025

blocked by #218147
@nickofthyme we need to wait first for the above PR to be merged on 8.x because this is a major bump (68 ->69) and updates the colors/style that where previously considered for Amsterdam to Borealis.

@markov00 markov00 removed the blocked label Apr 16, 2025
@markov00 markov00 requested a review from nickofthyme April 28, 2025 10:53
@markov00 markov00 changed the base branch from old-8.x to 8.19 April 28, 2025 14:01
@nickofthyme nickofthyme changed the title [8.x] Upgrade @elastic/charts to 69.2.1 (#216430) [8.19] Upgrade @elastic/charts to 69.2.1 (#216430) May 5, 2025
@markov00
Copy link
Contributor Author

markov00 commented May 7, 2025

currently blocked by this #220390 that brings in a fix ourside the elastic-charts code inthe SLO annotations (causing this PR to fail)

I've added the fixes from #220390 here 44f9c2d

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
ml 5.4MB 5.4MB +10.0B
observability 1.3MB 1.3MB +43.0B
uptime 494.6KB 494.6KB +30.0B
total +83.0B

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
charts 44.4KB 44.5KB +30.0B
kbnUiSharedDeps-npmDll 6.0MB 6.0MB -557.0B
observability 93.8KB 93.8KB -18.0B
total -545.0B

History

@markov00 markov00 marked this pull request as ready for review May 8, 2025 16:12
@markov00 markov00 merged commit ff0e0d5 into elastic:8.19 May 9, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants