Skip to content
This repository has been archived by the owner on Dec 10, 2021. It is now read-only.

Split @superset-ui/query from @superset-ui/chart #176

Closed
kristw opened this issue Jun 13, 2019 · 1 comment · Fixed by #178
Closed

Split @superset-ui/query from @superset-ui/chart #176

kristw opened this issue Jun 13, 2019 · 1 comment · Fixed by #178
Labels
#code-quality

Comments

@kristw
Copy link
Contributor

kristw commented Jun 13, 2019 •
edited
Loading

Is your feature request related to a problem? Please describe.

Logic for creating QueryContext and QueryObject are in the @superset-ui/chart package.

Describe the solution you'd like

Separate them into another package for simplicity.

Describe alternatives you've considered

Don't do anything.
@issue-label-bot
Copy link

Issue-Label Bot is automatically applying the label #enhancement to this issue, with a confidence of 0.97. Please mark this comment with 👍 or 👎 to give our bot feedback!

Links: app homepage, dashboard and code for this bot.

@issue-label-bot issue-label-bot bot added the #enhancement New feature or request label Jun 13, 2019
@kristw kristw added #code-quality and removed #enhancement New feature or request labels Jun 13, 2019
@kristw kristw mentioned this issue Jun 18, 2019
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
#code-quality
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: split superset-ui/query from superset-ui/chart apache-superset/superset-ui
1 participant
@kristw