-
Notifications
You must be signed in to change notification settings - Fork 30
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
Block Pattern: Post Grid Block #84
Comments
This might be easier to accomplish now with the Query Loop block likely capable for handling most of this. |
Hey @jeffpaul In my understanding Block Patterns are used for the static content blocks, In our case we need something dynamic to fetch the latest three post with the Podcast taxonomy. In this case we wont be able to use the Query Loop block also, Currently Query Loop block doesn't support custom taxonomies. It would say its still early for us to use Block Patterns in this case. We have a few open issues in Gutenberg repo regarding adding support for custom taxonomies. It would be worth waiting for it. WordPress/gutenberg#34977 Alternative, if we really want to ship something with Block Patterns then we need to develop a custom block to fetch latest posts with the Podcast Taxonomy. |
@ajmaurya99 thanks for the research notes, that's very helpful! I would say in general it would probably be best to wait until the Query Loop block (or some other core block) is able to help solve for this issue as the need for the Post Grid block for this plugin is fairly basic in nature and once CPTs work in the Query Loop block would easily cover our needs here. |
Wireframes for Post Grid Block: https://www.figma.com/file/14KjviZcFQZ1H6bIhSPwXa/Podcast-Block-Patterns?node-id=1%3A70 Credits: Nickolas Kola |
Note that the Query Loop block can likely handle this functionality now, so likely worth bumping this in priority on the roadmap: https://make.wordpress.org/core/2022/02/03/whats-new-in-gutenberg-12-5-february-2nd/#query-loop-custom-taxonomies-filtering-and-more |
Noting that @achchu93 is looking into this issue. |
As per the Figma design, there is a placeholder for duration. I identified going with a custom block for duration meta is the best practices. But now the question is, whether we gonna make a block which can pull the value with meta name or specifically fetch the value of duration meta. Need suggestion on this. |
@cadic any thoughts on ^ this question? |
@achchu93 the episode duration is stored in |
Since the stored value |
@achchu93 are you still working on a PR for this or should we unassign so someone else can pick up the issue? |
@jeffpaul Hey, I have completed all the functional tasks. Now we have left only the styling part. I can take a look at this start of next week if you allow me. |
FYI, potentially related: Extending the Query Loop block dev note from WP 6.1. |
@achchu93 could you please push up a branch or PR of your work, even if it's still in-progress? |
I have created a PR to add a block pattern for podcast grid. Let me know if there are any changes. Thanks! CC: @jeffpaul |
Is your enhancement related to a problem? Please describe.
With the advent of Block Patterns, we might want to consider a simple block pattern to cover the Post Grid Block concept.
Describe the solution you'd like
Designs
TBD, but ideal to help shape what we'd craft
Describe alternatives you've considered
n/a
Additional context
Block Pattern creation walkthroughs from the super rad @melchoyce.
The text was updated successfully, but these errors were encountered: