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

merge master #10

Merged
merged 13 commits into from
May 6, 2021
Merged

merge master #10

merged 13 commits into from
May 6, 2021

Conversation

stephenLYZ
Copy link
Owner

💔 Breaking Changes

🏆 Enhancements

📜 Documentation

🐛 Bug Fix

🏠 Internal

amitmiran137 and others added 13 commits May 3, 2021 21:41
* feat(plugin-chart-pivot-table): enable cross filtering

* Fix test

* Fix tests

* Replace `.toEqual` with `.toMatchObject` to fix test

* Use jest.fn() to mock setDataMask

* Code review changes

* Bring back setting ownState
* initial

* tests added,1 typing issue present

* remove log

* remove color

* remove duplicate tooltip

* Update plugins/plugin-chart-echarts/src/Tree/transformProps.ts

Co-authored-by: Ville Brofeldt <33317356+villebro@users.noreply.github.com>

* clean

* merge

* optional name column,more tests...

* bug fix

* remove unused imports

* lint

* bug fix

* tying fix

* clean,typing fix

* stories added

* optional metric control

* bigger thumbnail

* optional root node name

* storybokk updated

* children list seperated instead of mutating data,if root node not provided defaault it to node with most children

* remove logs,optimization needed

* find best node if root not provided,use seperate children list to avoid issues if column name is children

* remove unused import

* type fix

* Update transformProps.ts

* Update debugging.md

* better todo

* updated radio button with new signature

Co-authored-by: Ville Brofeldt <33317356+villebro@users.noreply.github.com>
@stephenLYZ stephenLYZ merged commit 023d3ed into stephenLYZ:master May 6, 2021
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.

7 participants