sneakers-the-rat / paper-feeds

A FastAPI web server for creating RSS feeds for scholarly journals with the magic of adversarial interoperability
GNU General Public License v3.0
63 stars 4 forks source link

some small improvements to the frontend #7

Closed lambdaloop closed 10 months ago

lambdaloop commented 10 months ago

the commits summarize the changes:

The last one is somewhat opinionated. I don't find the ISSN useful when checking the journal, but the recent paper count available on crossref instantly tells me if the feed is up to date. I wonder if we should just remove feeds with 0 recent papers, or get the latest papers from somewhere else since crossref is clearly not up to date??

lambdaloop commented 10 months ago

I also suggest removing the link in the journal title, especially if there is no page for it, but I haven't done that here.

sneakers-the-rat commented 10 months ago

Yes! The ISSN column was more of a debugging artifact to make sure that they were being fetched, so definitely an improvement, Lemme do a quick spot check

sneakers-the-rat commented 10 months ago

Looks good :)