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

Cherry-picks: Sunburst, Horizon, Treemap and a few small updates from community #101

Merged
merged 12 commits into from
Aug 27, 2018

Conversation

kristw
Copy link

@kristw kristw commented Aug 27, 2018

Cherry-pick the following commit(s):

SHA Time Log Author
8d01c84 Mon Aug 27 10:43:04 2018 -0700 [SIP-5] Refactor sunburst (apache#5699) Krist Wongsuphasawat
d7f06cb Mon Aug 27 10:42:42 2018 -0700 [SIP-5] Repair and refactor Horizon Chart (apache#5690) Krist Wongsuphasawat
f5cc983 Sun Aug 26 21:01:05 2018 -0700 remove unneeded console.log (apache#5738) Hugh A. Miles II
54ae215 Fri Aug 24 14:15:43 2018 -0700 [SIP-5] Refactor treemap (apache#5670) Krist Wongsuphasawat
5e6efae Fri Aug 24 06:48:26 2018 +0800 Geojson property names (apache#5708) Joshua Brooks
0448db3 Thu Aug 23 15:46:32 2018 -0700 remove console.log and change wording (apache#5702) Krist Wongsuphasawat
9fb28b5 Thu Aug 23 15:45:45 2018 -0700 Revise markup.js and iframe.js (apache#5672) Krist Wongsuphasawat
0e057d5 Thu Aug 23 00:26:15 2018 +0800 Adds geodata for Myanmar (apache#5697) Joshua Brooks
1c85192 Thu Aug 23 00:25:57 2018 +0800 adds TimorLeste geodata (apache#5698) Joshua Brooks
dfb6147 Wed Jul 25 11:02:50 2018 -0700 Remove is-react dep (apache#5478) Beto Dealmeida

Pass js unit test and verified on development box.

@john-bodley @conglei @graceguo-supercat @kristw @michellethomas @timifasubaa @williaster

betodealmeida and others added 11 commits August 27, 2018 13:38
* Add geojson for Myanmar

* Adds 'myanmar.geojson' file

* Adds "iso-3166" codes

(cherry picked from commit 0e057d5)
* Do not call slice.xxx

* remove iframe id

* remove old code

* use import instead of require

* update iframe.js

(cherry picked from commit 9fb28b5)
* Add geojson for Myanmar

* Adds 'myanmar.geojson' file

* Adds "iso-3166" codes

* Uppercase ISO fieldname in TL, MY geodata

(cherry picked from commit 5e6efae)
* refactor treemap

* refactor treemap code

* Add proptypes

* add proptypes for tree

* Add margin prop

(cherry picked from commit 54ae215)
* Migrate horizon chart to react

* remove unused code

* rename files

* update props

* enable renderTrigger

* fix canvas transform issue

* Address Chris' comment

(cherry picked from commit d7f06cb)
* extract slice and formData

* fix small issues

* Update visual

* update css

(cherry picked from commit 8d01c84)
@kristw kristw changed the title [WIP] Cherry-pick(s): <description> Cherry-picks: Sunburst, Horizon, Treemap and a few small updates from community Aug 27, 2018
@kristw kristw requested review from graceguo-supercat and williaster and removed request for graceguo-supercat August 27, 2018 20:48
@codecov-io
Copy link

codecov-io commented Aug 27, 2018

Codecov Report

Merging #101 into airbnb-master will decrease coverage by <.01%.
The diff coverage is 2.02%.

Impacted file tree graph

@@                Coverage Diff                @@
##           airbnb-master     #101      +/-   ##
=================================================
- Coverage          63.19%   63.18%   -0.01%     
=================================================
  Files                351      352       +1     
  Lines              22055    22054       -1     
  Branches            2473     2470       -3     
=================================================
- Hits               13937    13935       -2     
- Misses              8106     8107       +1     
  Partials              12       12
Impacted Files Coverage Δ
superset/assets/src/chart/Chart.jsx 57.69% <ø> (+0.22%) ⬆️
superset/assets/src/visualizations/word_cloud.js 0% <ø> (ø) ⬆️
superset/assets/src/explore/controls.jsx 45.8% <ø> (ø) ⬆️
...uperset/assets/src/visualizations/HorizonChart.jsx 0% <0%> (ø)
superset/assets/src/visualizations/treemap.js 0% <0%> (ø) ⬆️
superset/assets/src/visualizations/iframe.js 0% <0%> (ø) ⬆️
superset/assets/src/visualizations/sunburst.js 0% <0%> (ø) ⬆️
superset/assets/src/visualizations/markup.js 0% <0%> (ø) ⬆️
superset/assets/src/visualizations/HorizonRow.jsx 0% <0%> (ø)
.../src/explore/components/ControlPanelsContainer.jsx 88.31% <100%> (ø) ⬆️
... and 3 more

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 f06365d...f95bf14. Read the comment docs.

@kristw kristw merged commit 424ddce into airbnb:airbnb-master Aug 27, 2018
@kristw kristw deleted the kristw-cherry-aug27v2 branch August 28, 2018 17:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants