You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by GitBox <gi...@apache.org> on 2018/07/24 00:57:35 UTC
[GitHub] mistercrunch commented on issue #5356: added a 'no_trend_line'
option
mistercrunch commented on issue #5356: added a 'no_trend_line' option
URL: https://github.com/apache/incubator-superset/pull/5356#issuecomment-407245558
@williaster this one will take the latest available dot in the time series where "Big Number" would disregard the time granularity.
Personally I'm ok with a revert on this if you prefer. I know you had touched `big_number.js` in the past and there's an open PR on this. Happy to defer to that.
I'll make sure to keep PRs open longer before merging them to allow for more input.
----------------------------------------------------------------
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