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

Latest Posts: changing number of posts doesn't refresh #1991

Closed
mtias opened this issue Jul 24, 2017 · 6 comments
Closed

Latest Posts: changing number of posts doesn't refresh #1991

mtias opened this issue Jul 24, 2017 · 6 comments
Labels
[Feature] Blocks Overall functionality of blocks [Type] Bug An existing feature does not function as intended

Comments

@mtias
Copy link
Member

mtias commented Jul 24, 2017

This is a regression. Changing number of posts doesn't update the block until you save and reload the post.

@mtias mtias added [Feature] Blocks Overall functionality of blocks [Type] Bug An existing feature does not function as intended labels Jul 24, 2017
@westonruter westonruter self-assigned this Jul 24, 2017
@westonruter
Copy link
Member

Taking a look…

@mtias
Copy link
Member Author

mtias commented Jul 24, 2017

Looks like postToShowCurrent === postToShowNext is returning true.

@westonruter
Copy link
Member

The regression was introduced as part of #1889. /cc @aduth

@aduth
Copy link
Member

aduth commented Jul 24, 2017

I'm not able to reproduce. Can we be more specific with steps, or what of #1889 caused the regression?

@westonruter
Copy link
Member

westonruter commented Jul 24, 2017

@aduth I used git-bisect to find the commit that caused the problem. It was among the commits in #1889. Specifically, a058493 first made it so that that number field couldn't be changed at all, and then the following commit 0f6a591 allowed the field to be modified again, but then the changes to the field didn't cause any changes to be reflected in the component. At the previous commit 012c0ee, the first commit in the PR, the posts shown field is editable and its changes are reflected in the block.

@mtias
Copy link
Member Author

mtias commented Jul 24, 2017

It is fixed after #1995

@aduth aduth closed this as completed Jul 24, 2017
Tug pushed a commit that referenced this issue Mar 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Blocks Overall functionality of blocks [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

No branches or pull requests

3 participants