Skip to content

Individual Episode Cover Art#273

Merged
jeffpaul merged 10 commits intodevelopfrom
feature/episode-cover-art
Mar 1, 2024
Merged

Individual Episode Cover Art#273
jeffpaul merged 10 commits intodevelopfrom
feature/episode-cover-art

Conversation

@zamanq
Copy link
Copy Markdown
Contributor

@zamanq zamanq commented Jan 26, 2024

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:

  • I have read the CONTRIBUTING document.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my change.
  • All new and existing tests passed.

Changelog Entry

Added - Ability to add Unique Cover Art for Episodes

Credits

Props @zamanq @jeffpaul

@zamanq zamanq requested a review from a team as a code owner January 26, 2024 19:41
@zamanq zamanq requested review from iamdharmesh and removed request for a team January 26, 2024 19:41
@github-actions github-actions bot added this to the 1.8.0 milestone Jan 26, 2024
@zamanq zamanq requested a review from jeffpaul January 26, 2024 19:42
@github-actions github-actions bot added the needs:code-review This requires code review. label Jan 26, 2024
@iamdharmesh
Copy link
Copy Markdown
Member

Hi @zamanq,

Thank you for the PR.

Right now user can use the default featured image as episode cover art and that'd also render the same metadata in the feed.

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

@zamanq
Copy link
Copy Markdown
Contributor Author

zamanq commented Feb 6, 2024

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 podcast_episode_cover for this custom setting is different than the default featured image. The featured image will get priority when rendering the image in the feed and if not set then it'll look for the new custom cover art.

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
cc: @jeffpaul

Thanks!

@jeffpaul
Copy link
Copy Markdown
Member

@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?

@iamdharmesh
Copy link
Copy Markdown
Member

@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 <itunes:image href='url-of-the-image' />). However, this PR adds a new image select field on the sidebar settings for the block to make it easier for the user.

So, now we have two fields for the episode cover image:

  1. Featured image
  2. Episode cover image field in the block sidebar.

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.

@jeffpaul
Copy link
Copy Markdown
Member

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?

@zamanq
Copy link
Copy Markdown
Contributor Author

zamanq commented Feb 25, 2024

@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.

Copy link
Copy Markdown
Member

@iamdharmesh iamdharmesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the changes @zamanq. Added 2 minor comments/suggestions. otherwise, looks good.

Thank you.

Copy link
Copy Markdown
Member

@iamdharmesh iamdharmesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

Thanks for working on this @zamanq.

@jeffpaul jeffpaul merged commit 301b6d1 into develop Mar 1, 2024
@jeffpaul jeffpaul deleted the feature/episode-cover-art branch March 1, 2024 18:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs:code-review This requires code review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unique Cover Art for Episodes

3 participants