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

fix: reenable table chart page length #9884

Merged
merged 1 commit into from
May 22, 2020

Conversation

ktmud
Copy link
Member

@ktmud ktmud commented May 22, 2020

SUMMARY

Bump @superset-ui/legacy-preset-chart-table to fix missing
page length support.

See apache-superset/superset-ui#522

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

N/A

TEST PLAN

Added unit-test in superset-ui.

Manual testing:

  • Go to a table chart
  • Change page length in Customize
  • Make sure pagination still works

ADDITIONAL INFORMATION

  • Has associated issue:
  • Changes UI
  • Requires DB Migration.
  • Confirm DB Migration upgrade and downgrade tested.
  • Introduces new feature or API
  • Removes existing feature or API

Bump @superset-ui/legacy-preset-chart-table to fix disabled
page length support.

See apache-superset/superset-ui#522
@codecov-commenter
Copy link

codecov-commenter commented May 22, 2020

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #9884   +/-   ##
=======================================
  Coverage   71.24%   71.24%           
=======================================
  Files         584      584           
  Lines       30789    30789           
  Branches     3234     3234           
=======================================
+ Hits        21936    21937    +1     
+ Misses       8744     8743    -1     
  Partials      109      109           
Flag Coverage Δ
#cypress 53.84% <ø> (+0.01%) ⬆️
#javascript 59.44% <ø> (ø)
#python 71.39% <ø> (ø)
Impacted Files Coverage Δ
.../src/dashboard/components/gridComponents/Chart.jsx 87.64% <0.00%> (+1.12%) ⬆️

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 ee99196...3d07fe9. Read the comment docs.

@john-bodley john-bodley merged commit 6d4e236 into apache:master May 22, 2020
@ktmud ktmud deleted the bugfix-table-page-length branch May 22, 2020 17:36
john-bodley pushed a commit to airbnb/superset-fork that referenced this pull request May 22, 2020
Bump @superset-ui/legacy-preset-chart-table to fix disabled
page length support.

See apache-superset/superset-ui#522

(cherry picked from commit 6d4e236)
pkdotson pushed a commit to preset-io/superset that referenced this pull request May 26, 2020
Bump @superset-ui/legacy-preset-chart-table to fix disabled
page length support.

See apache-superset/superset-ui#522
auxten pushed a commit to auxten/incubator-superset that referenced this pull request Nov 20, 2020
Bump @superset-ui/legacy-preset-chart-table to fix disabled
page length support.

See apache-superset/superset-ui#522
@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