Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: bump superset-ui/time-format and big-number #9808

Merged
merged 2 commits into from
May 18, 2020

Conversation

ktmud
Copy link
Member

@ktmud ktmud commented May 14, 2020

SUMMARY

Bump superset-ui/time-format and the big number chart to add full dates to big number datetime formatted in weekly interval (and remove week numbers).

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

See apache-superset/superset-ui#486
and apache-superset/superset-ui#493

TEST PLAN

Unit tests were added to superset-ui.

ADDITIONAL INFORMATION

@ktmud ktmud force-pushed the bignumber-tooltip-format branch from 482de02 to 3b0aa77 Compare May 14, 2020 20:14
@ktmud ktmud force-pushed the bignumber-tooltip-format branch from 3b0aa77 to d205de6 Compare May 15, 2020 05:12
@codecov-io
Copy link

codecov-io commented May 15, 2020

Codecov Report

Merging #9808 into master will increase coverage by 0.13%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #9808      +/-   ##
==========================================
+ Coverage   70.89%   71.02%   +0.13%     
==========================================
  Files         585      184     -401     
  Lines       30427    17863   -12564     
  Branches     3152        0    -3152     
==========================================
- Hits        21572    12688    -8884     
+ Misses       8744     5175    -3569     
+ Partials      111        0     -111     
Flag Coverage Δ
#cypress ?
#javascript ?
#python 71.02% <ø> (ø)
Impacted Files Coverage Δ
superset/views/core.py 75.13% <0.00%> (ø)
superset/datasets/commands/exceptions.py 92.15% <0.00%> (ø)
superset-frontend/src/explore/store.js
...end/src/dashboard/util/getKeyForFilterScopeTree.js
superset-frontend/src/dashboard/util/constants.js
superset-frontend/src/utils/hostNamesConfig.js
superset-frontend/src/logger/LogUtils.js
...src/explore/components/controls/SpatialControl.jsx
...end/src/SqlLab/components/ExploreResultsButton.jsx
superset-frontend/src/components/Menu/UserMenu.jsx
... and 393 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 34df11a...ba43371. Read the comment docs.

ktmud added 2 commits May 18, 2020 09:55
This is for adding full dates to big number datetime formatted
by weekly interval.
@ktmud ktmud force-pushed the bignumber-tooltip-format branch from ba43371 to a70a592 Compare May 18, 2020 17:13
@kristw kristw merged commit c4016e3 into apache:master May 18, 2020
@ktmud ktmud deleted the bignumber-tooltip-format branch May 18, 2020 19:52
auxten pushed a commit to auxten/incubator-superset that referenced this pull request Nov 20, 2020
* feat: bump superset-ui/time-format and big-number

This is for adding full dates to big number datetime formatted
by weekly interval.

* Bump time-format to v0.13.15

to bring in apache-superset/superset-ui#493
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.37.0 labels Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels size/XS 🚢 0.37.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants