-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
add itunes podcast icon #2114
add itunes podcast icon #2114
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @HyShai, makes sense to me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice idea! 👍
@@ -296,6 +296,7 @@ def receive_web_request(params, method, format) | |||
<atom:link href=#{feed_url(secret: params['secret'], format: :xml).encode(xml: :attr)} rel="self" type="application/rss+xml" /> | |||
<atom:icon>#{feed_icon.encode(xml: :text)}</atom:icon> | |||
#{hub_links} | |||
<itunes:image href=#{feed_icon.encode(xml: :attr)} /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this should be added only if boolify(interpolated['ns_itunes'])
is true, because it is the option to enable the itunes
namespace.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yup that makes sense - changes have been implemented
@knu Yup makes sense - I've implemented those changes... |
👍 |
No description provided.