Conversation
|
Hi @zamanq, Thank you for the PR.
I started reviewing the PR and am curious about the new "Add Cover Art" button. Currently, the default featured image is already being used as cover art for the episode in the feed. Have we added this button within the podcast block for better visibility and to avoid confusion, or is there another specific reason behind it? Thanks |
|
Hi @iamdharmesh, Yes the goal was to provide an easy way for the user to set cover art individually for an episode while being on the sidebar settings for the block. The meta key As a whole this seems redundant having two different options for achieving the same thing. Actually, this is due to my oversight of not inspecting the whole thing first and was really focused on creating that feature assuming that it is not introduced yet as the issue was reported in #255 Thanks! |
|
@iamdharmesh are you saying that if the post that the podcast is attached to has a featured image that that image gets sent as the episode image versus otherwise inheriting the default show image? |
@jeffpaul No. Currently, the plugin has already sent a featured image as an episode image (by setting So, now we have two fields for the episode cover image:
I would suggest checking the possibility of showing the featured image in the block sidebar, similar to what we have done for the podcast taxonomy terms (PR #183). If that is possible, we can achieve this without adding additional metadata in the database. If this is not possible at the moment, then we are good with PR. |
I agree, that sounds like the most ideal. @zamanq can you research and try that if its feasible? |
…dcasting into feature/episode-cover-art
|
@jeffpaul @iamdharmesh Refactored the code to leverage the post featured image as the episode cover art. If there's none, the user can select one either using the default post featured image area or using the block sidebar and it'll reflect in the featured image as well. |
iamdharmesh
left a comment
There was a problem hiding this comment.
Thanks for the changes @zamanq. Added 2 minor comments/suggestions. otherwise, looks good.
Thank you.
…dcasting into feature/episode-cover-art
iamdharmesh
left a comment
There was a problem hiding this comment.
LGTM 🚀
Thanks for working on this @zamanq.
Description of the Change
Added the option to select cover art for individual episodes. Right now user can use the default featured image as episode cover art and that'd also render the same metadata in the feed. With this new option they can also add an image from the Podcast Settings panel. There's a new button "Add Cover Art" which upon clicking will open the media modal. After selection user will be able to replace or remove the media as well.
So if there's no featured image selected then the new custom cover art will get used and vice-versa.
Closes #255
Verification Process
Upon selecting a cover art, it'll be available in the feed channel url under item along with all the other item feed properties. The tag will be
<itunes:image href='url-of-the-image' />Checklist:
Changelog Entry
Credits
Props @zamanq @jeffpaul