[GitHub] mistercrunch opened a new pull request #5682: [bugfix] making secondary_metric optional

[GitBox <gi...@apache.org>]

mistercrunch opened a new pull request #5682: [bugfix] making secondary_metric optional
URL: https://github.com/apache/incubator-superset/pull/5682
 
 
   I noticed that `secondary_metric` was required in the chart that uses
   it, namely sunburst, parallel coordinates, and world_map.
   
   I set the control as allowing null, and found related bugs in the
   process:
   * parallel coordinates did not have support for the new MetricControl
   * added color scheme support for parallel coordinates
   * added option to set row_limit on these viz types
   * sunburst to support numeric columns (number would show as blank)

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org