- πΊπΈUnited States dww
FWIW, +1 to this issue. Echoing #13, I'm validating a feed built with this module at https://validator.w3.org/feed/check.cgi and getting this:
This feed does not validate.
line 2, column 1018: Undefined channel element: author [help]The help link is a page that says:
Your feed contains an element that this validator does not recognize in this context.
I think the W3 RSS validator should be considered fairly authoritative for a "standard spec" on RSS feeds...
I looked briefly at the MR, which mostly looks good. Since π Missing schema for views style and row Fixed just landed (yay! π), we need to update the config schema to match the renamed option here.
- πͺπΈSpain pcambra Asturies
Author mentioned on #12 refers to the item, and we're removing it from the channel, so I think we're good.
We already have an itunes:author_field, so removing the other one rather than renaming it. - Assigned to pcambra
- Status changed to Fixed
9 months ago 6:28pm 27 March 2024 - πͺπΈSpain pcambra Asturies
I messed up the commit message :(, apologies about that, I've tried to rename the commit but haven't been able because it's tied to a release. Wondering if I should revert and re-commit with the nice commit message I had before squashing in the gitlab ui?
- πΊπΈUnited States dww
Mistakes happen, no worries. π Yeah, the main repos do not allow forced push. I suppose a revert and a re-commit with a proper message would work well, if you're up for it.
Thanks again!
-Derek -
pcambra β
committed 33aa2250 on 8.x-1.x
Issue #3257463 by elber, pcambra, rudolfbyker, dww: Remove <author>
-
pcambra β
committed 33aa2250 on 8.x-1.x
Automatically closed - issue fixed for 2 weeks with no activity.