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

Welcome page pagination is inconsistent #10232

Closed
etr2460 opened this issue Jul 2, 2020 · 6 comments · Fixed by #9964
Closed

Welcome page pagination is inconsistent #10232

etr2460 opened this issue Jul 2, 2020 · 6 comments · Fixed by #9964
Labels
!deprecated-label:bug Deprecated label - Use #bug instead

Comments

@etr2460
Copy link
Member

etr2460 commented Jul 2, 2020

The pagination of dashboards on the welcome page doesn't have a consistent number of dashboards per page, and the record count isn't sequential. There record counts also seem to flicker and update to different numbers as the page loads (e.g. displaying 1-11, then updating to 1-15 once items render)

Expected results

Each page to have the same number of items, record counting to be sequential (e.g. first page is 1-10, 2nd is 11-20, 3rd is 21-30, etc)

Screenshots

Screen Shot 2020-07-02 at 8 49 25 AM

Screen Shot 2020-07-02 at 8 49 33 AM

How to reproduce the bug

Paginate on the welcome page

Environment

(please complete the following information):

  • superset version: master

We have not yet enabled React List Views in our environment

to: @nytai, I think you were working on this?

@etr2460 etr2460 added the !deprecated-label:bug Deprecated label - Use #bug instead label Jul 2, 2020
@issue-label-bot
Copy link

Issue-Label Bot is automatically applying the label #bug to this issue, with a confidence of 0.94. Please mark this comment with 👍 or 👎 to give our bot feedback!

Links: app homepage, dashboard and code for this bot.

@nytai
Copy link
Member

nytai commented Jul 2, 2020

@etr2460 I have noticed some weirdness with the record counts being inconsistent, I'll look into that now. To be clear, you have "whited-out" the dashboard and creator columns in the screenshots and that isn't a bug?

@bkyryliuk
Copy link
Member

I've gotten similar bug reports from our users.

@nytai
Copy link
Member

nytai commented Jul 2, 2020

I'm also confused where the 10 records per page is coming from. In the code it's currently set to 25, https://github.com/preset-io/incubator-superset/blob/master/superset-frontend/src/welcome/DashboardTable.jsx#L28

@dpgaspar
Copy link
Member

dpgaspar commented Jul 2, 2020

I think this may be related to a bug I currently fixed on FAB related with m-m relations on lists. The amount of owners are counted has records also, will publish an FAB 3 RC2 tomorrow if it's not too urgent

@etr2460
Copy link
Member Author

etr2460 commented Jul 5, 2020

@nytai: correct, I whited out the dashboard and creator columns to prevent exposing sensitive data.

@dpgaspar: thanks for the fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
!deprecated-label:bug Deprecated label - Use #bug instead
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants