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.

Sign up for GitHub

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

Jump to bottom

feat(chart): Added optional id to data #34

Merged

SanjeevLakhwani merged 1 commit into main from data-id

Nov 6, 2024

+1 −0

Collaborator

SanjeevLakhwani commented Nov 6, 2024

This id is copied over to onClick responses by recharts which helps recognize keys that are different from title such as translation.


          feat(chart): Added optional id to data

798e02e

SanjeevLakhwani requested a review from v-rocheleau

November 6, 2024 18:25

SanjeevLakhwani self-assigned this

v-rocheleau approved these changes

View reviewed changes

Contributor

v-rocheleau left a comment

lgtm

SanjeevLakhwani merged commit a2daee4 into main

1 check passed

SanjeevLakhwani deleted the data-id branch

November 6, 2024 18:30

github-actions bot pushed a commit that referenced this pull request


          chore(release): 2.10.0 [skip ci]

# [2.10.0](v2.9.3...v2.10.0) (2024-11-06)

### Features

* **chart:** Added optional id to data ([798e02e](798e02e))
* **chart:** Added optional id to data ([#34](#34)) ([a2daee4](a2daee4))

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet