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

Upgrade antd and icons #2576

Open
yurishkuro opened this issue Jan 9, 2025 · 1 comment
Open

Upgrade antd and icons #2576

yurishkuro opened this issue Jan 9, 2025 · 1 comment

Comments

@yurishkuro
Copy link
Member

Automatic upgrades are failing

There are probably some minor API changes. But since this affects the visualization, a fix should include before/after screenshots of the changed components, to verify continuity.

@141AQUA-xzy
Copy link

i would like to work on this issue

yurishkuro pushed a commit that referenced this issue Jan 10, 2025
## Which problem is this PR solving?
Issue #2576


## Description of the changes
* Antd moved from tinycolor to fast-color in their new minor changes
ant-design/ant-design#52107
* I removed/uninstall the @ctrl/tinycolor module and re-install the antd
to latest version as per this
[docs](https://ant.design/docs/react/migration-v5)

## How was this change tested?
* automated testing
* some changes to snapshot (npm run test-ci -- -u) 

## Checklist
- [x] I have read
https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
- [x] I have signed all commits
- [x] I have added unit tests for the new functionality
- [x] I have run lint and test steps successfully
  - for `jaeger`: `make lint test`
  - for `jaeger-ui`: `npm run lint` and `npm run test`

---------

Signed-off-by: Rahul Vishwakarma <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants