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

[ LatestPosts ] Move the new multiple category selector to QueryControls #20826

Closed
draganescu opened this issue Mar 12, 2020 · 1 comment
Closed
Labels
[Block] Latest Posts Affects the Latest Posts Block [Type] Bug An existing feature does not function as intended

Comments

@draganescu
Copy link
Contributor

Describe the bug
In #20781 we added a multiple select control to the LatestPosts block which allows filtering the posts by multiple categories.

However, the control is being used directly in the block instead of the reusable QueryControls component. Given that the block itself is to be refactored at some time in the future to use the new QueryBlock it is important that we keep the filtering together in a component so we can more easily migrate when the time comes.

Expected behavior
The new filter should be in the QueryControls component.

@draganescu draganescu changed the title [ LatestPosts ]Move the new multiple category selector to QueryControls [ LatestPosts ] Move the new multiple category selector to QueryControls Mar 12, 2020
@talldan talldan added [Block] Latest Posts Affects the Latest Posts Block [Type] Bug An existing feature does not function as intended labels Mar 16, 2020
@draganescu
Copy link
Contributor Author

Solved in #20832

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Latest Posts Affects the Latest Posts Block [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

No branches or pull requests

2 participants