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

md-table's data doesn't get updated properly #1492

Closed
ueharajohji opened this issue Feb 8, 2018 · 3 comments · Fixed by #1496
Closed

md-table's data doesn't get updated properly #1492

ueharajohji opened this issue Feb 8, 2018 · 3 comments · Fixed by #1496
Labels

Comments

@ueharajohji
Copy link

ueharajohji commented Feb 8, 2018

Steps to reproduce

Which browser?

What is expected?

What is actually happening?

I'm trying to implement a pagination function with vue 2.5.3 / vue-material1.0.0 and the problem is whenever I update the
model , the table gets updated with the newly fetched model but the md-table component still has the previous data so when I check the "select all" box it gets the previous data only which makes the selected variable contain only the page 1 data when in fact I am in page 2. anyone have any suggestions on how to deal with this?

Reproduction Link

@VdustR
Copy link
Member

VdustR commented Feb 8, 2018

Would you provide a reproduction for this?

@ueharajohji
Copy link
Author

Sorry for the late reply ,

here is the reproduction , I forked your codepen a while ago and then tweaked it to
reproduce the problem I am experiencing right now.

https://codepen.io/ueharajohji/pen/NybVMy

to reproduce the bug

Just simply click the next page button and check the select all box, and you will see
even though I already replaced the people array with just 1 data ( dummy ) it still gets
the previous data.

@ueharajohji
Copy link
Author

Thank you @VdustR ! I will inform you about my progress on the pagination once I applied your patch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants