Skip to content

Conversation

@aaxelb
Copy link
Contributor

@aaxelb aaxelb commented Oct 24, 2025

fix validation errors found on index-card-search rss/atom feeds

  • atom renderer (see validation report, now fixed)
    • <link href="..." /> instead of <link>...</link>
    • add <updated> feed element
    • fix missing <author>
    • only one <uri> per <author>
    • include filename as additional title
  • rss renderer (see validation report, now fixed)
    • use dc:creator instead of author (which can occur only once and requires email address)
    • use filename for title when title missing
    • add self-link <atom:link rel="self" ...
  • also add feed links to plain json rendering

@aaxelb aaxelb requested a review from felliott October 24, 2025 15:53
@coveralls
Copy link

coveralls commented Oct 24, 2025

Coverage Status

coverage: 84.4% (+0.1%) from 84.29%
when pulling 8d48649 on aaxelb:feat/9046/shtrove-rss-followup
into 71ebed7 on CenterForOpenScience:develop.

Copy link
Member

@felliott felliott left a comment

Choose a reason for hiding this comment

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

Seems reasonable to me. Approved!

Cheers,
@felliott

@aaxelb aaxelb merged commit c35c715 into CenterForOpenScience:develop Oct 28, 2025
3 checks passed
@aaxelb aaxelb deleted the feat/9046/shtrove-rss-followup branch October 28, 2025 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants