Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

[cherry-pick] Cherry-picking w/ overrides #42

Merged

Conversation

john-bodley
Copy link

This PR cherry picks apache#4835 which remedies the broken build. Note I had to manually resolve cherry-pick conflicts as the domain of dependence was becoming unwieldy as I tried to find the subset of upstream PRs which were completely decoupled.

to: @graceguo-supercat @michellethomas @timifasubaa

@codecov-io
Copy link

codecov-io commented Apr 20, 2018

Codecov Report

Merging #42 into airbnb-master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@              Coverage Diff               @@
##           airbnb-master      #42   +/-   ##
==============================================
  Coverage          71.14%   71.14%           
==============================================
  Files                188      188           
  Lines              14849    14849           
  Branches            1085     1085           
==============================================
  Hits               10564    10564           
  Misses              4282     4282           
  Partials               3        3
Impacted Files Coverage Δ
superset/cli.py 46.83% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bd1273a...ee3c8f3. Read the comment docs.

@john-bodley
Copy link
Author

ping @michellethomas @timifasubaa

@timifasubaa
Copy link

LGTM

@john-bodley john-bodley merged commit f667151 into airbnb:airbnb-master Apr 23, 2018
@john-bodley john-bodley deleted the john-bodley-cherry-pick-4835 branch April 23, 2018 22:50
michellethomas pushed a commit that referenced this pull request Dec 1, 2021
…#42)

Tap into the newly added `willUnmount` hook to clean up sticky tooltip that nvd3 adds to `body` directly. This removes such tooltips that were left behind in the explore view with the new query returns 'no data'.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants