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

Cherrypick a few dash fixes #86

Conversation

graceguo-supercat
Copy link
Collaborator

@graceguo-supercat graceguo-supercat commented Aug 14, 2018

this PR cherrypick following prs from Superset master:

Dashboard related:
[Dashobard]Fix bad merge (apache#5624)
set default layout for new added charts (apache#5425)
Remove dashboard transition related config (apache#5505)

This feature improves line type for timeshift chart:
Improve time shift (apache#5140)

@michellethomas @john-bodley @timifasubaa

betodealmeida and others added 4 commits August 14, 2018 11:37
* Improve time shift color and pattern

* Revert change

* Fix js unit test

* Move code to better place, add unit test

* Move classed code to backend

* Remove console.log

* Remove 1 hour time compare

* Remove unused import

(cherry picked from commit 57e1256)
(cherry picked from commit 9f6ac08)
@codecov-io
Copy link

codecov-io commented Aug 14, 2018

Codecov Report

Merging #86 into airbnb-master will decrease coverage by 0.02%.
The diff coverage is 37.5%.

Impacted file tree graph

@@               Coverage Diff                @@
##           airbnb-master     #86      +/-   ##
================================================
- Coverage          63.23%   63.2%   -0.03%     
================================================
  Files                351     351              
  Lines              22037   22043       +6     
  Branches            2463    2465       +2     
================================================
- Hits               13935   13933       -2     
- Misses              8090    8098       +8     
  Partials              12      12
Impacted Files Coverage Δ
superset/views/core.py 73.68% <ø> (-0.02%) ⬇️
superset/config.py 92.42% <ø> (-0.23%) ⬇️
...t/assets/src/dashboard/reducers/getInitialState.js 0% <0%> (ø) ⬆️
superset/assets/src/utils/common.js 60.52% <100%> (+0.52%) ⬆️
superset/assets/src/modules/colors.js 77.55% <100%> (+0.46%) ⬆️
superset/assets/src/dashboard/util/constants.js 100% <100%> (ø) ⬆️
superset/assets/src/visualizations/nvd3_vis.js 9.03% <100%> (-0.39%) ⬇️
...t/assets/src/dashboard/util/newComponentFactory.js 100% <100%> (ø) ⬆️
superset/viz.py 80.71% <40%> (-0.09%) ⬇️

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 de73024...8dd7184. Read the comment docs.

@graceguo-supercat graceguo-supercat merged commit c3f33ce into airbnb:airbnb-master Aug 14, 2018
@graceguo-supercat graceguo-supercat deleted the gg-CherrypickDashFixes branch August 14, 2018 21:48
michellethomas pushed a commit that referenced this pull request Dec 1, 2021
…cy (#86)

* feat: add new subgenerators for legacy package and demo
* feat: make the default generator provide choices to choose subgenerators
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.

4 participants