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

Viz type #37

Merged
merged 3 commits into from
Oct 2, 2015
Merged

Viz type #37

merged 3 commits into from
Oct 2, 2015

Conversation

mistercrunch
Copy link
Member

@akuhn, a few changes we spoke about and a bugfix

@mistercrunch
Copy link
Member Author

BTW I pushed to master a fix for the False attribution to checkboxes and all. The key was to build the URL using the form_data before it's digested by the form...

@@ -248,6 +252,7 @@ def get_json_data(self):


class NVD3Viz(BaseViz):
viz_type = None
verbose_name = "Base NVD3 Viz"
template = 'panoramix/viz_nvd3.html'
chart_kind = 'line'
Copy link
Contributor

Choose a reason for hiding this comment

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

Guess you can also remove this.

@akuhn
Copy link
Contributor

akuhn commented Oct 1, 2015

 LGTM

mistercrunch added a commit that referenced this pull request Oct 2, 2015
@mistercrunch mistercrunch merged commit f3c6414 into master Oct 2, 2015
@mistercrunch mistercrunch deleted the viz_type branch October 2, 2015 00:49
hughhhh referenced this pull request in lyft/incubator-superset Apr 1, 2018
Make sure upstream folder is cloned properly
NOT PASSING TESTS
betodealmeida pushed a commit that referenced this pull request Sep 20, 2019
graceguo-supercat pushed a commit to graceguo-supercat/superset that referenced this pull request Oct 4, 2021
BREAKING CHANGE: Change Registry constructor API to take object instead of single string name.
feat: Add overwritePolicy for Registry so developer can customize whether overwriting is ALLOW, WARN or PROHIBIT.
zhaoyongjie pushed a commit that referenced this pull request Nov 24, 2021
BREAKING CHANGE: Change Registry constructor API to take object instead of single string name.
feat: Add overwritePolicy for Registry so developer can customize whether overwriting is ALLOW, WARN or PROHIBIT.
zhaoyongjie pushed a commit that referenced this pull request Nov 24, 2021
* fix: change references to vega-lite that broke build

* refactor: rename fielddef to channeldef
zhaoyongjie pushed a commit that referenced this pull request Nov 25, 2021
BREAKING CHANGE: Change Registry constructor API to take object instead of single string name.
feat: Add overwritePolicy for Registry so developer can customize whether overwriting is ALLOW, WARN or PROHIBIT.
zhaoyongjie pushed a commit that referenced this pull request Nov 25, 2021
* fix: change references to vega-lite that broke build

* refactor: rename fielddef to channeldef
zhaoyongjie pushed a commit that referenced this pull request Nov 26, 2021
BREAKING CHANGE: Change Registry constructor API to take object instead of single string name.
feat: Add overwritePolicy for Registry so developer can customize whether overwriting is ALLOW, WARN or PROHIBIT.
zhaoyongjie pushed a commit that referenced this pull request Nov 26, 2021
* fix: change references to vega-lite that broke build

* refactor: rename fielddef to channeldef
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.5.0 labels Feb 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.5.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants