Evan Rusackas

Results 1381 comments of Evan Rusackas

Thanks for the approval, but this is only the tip of the iceberg :) This found (if memory serves) something like 34 links that need to be addressed. These will...

I mean... it seems like it'll let us merge with the test failing as-is, but that seems like bad hygiene for other PRs that follow before I clean up my...

Upgrading npm and node will likely be part of Superset 5.0. We just upgraded it, so we're letting that soak before we do it again. I _think_ core-js is a...

The tooltip looks strangely high up, otherwise we're looking good, and thanks for the touchup! I'll try to find time in a bit to pull your branch and see about...

OK, I think we're good to go otherwise... I'll await feedback before unilaterally merging. I think Truthfulness outweighs Consitency, which outweighs pretty much any other random change ;)

Any reproduction steps whatsoever would help someone understand what you're up against. We don't know how you're deploying Superset, how you're embedding it, what role you're using for the guest...

Ahh yes.. the crossfilter actually changes the data being fed to the table, so the total is calculated. The filter IN the table doesn't appear to recalculate on the fly....

Hmm... I can't play the video for some reason. :/

I think this one actually needs an import change before merging

@mistercrunch fixed! (cc @michael-s-molina since we discussed this earlier)