internetarchive / fatcat-scholar

search interface for scholarly works
https://scholar.archive.org
Other
80 stars 14 forks source link

Display volume, issue, pages #53

Open bnewbold opened 3 years ago

bnewbold commented 3 years ago

In the "journal" row, after the journal name. Not sure what the style should be, eg:

Vol. 1, Iss. 2, p3-4

Or full words?

Volume 1, Issue 2, p3-4

Or citation style?

1(2) p3-4

dwaipayan05 commented 3 years ago

Hi @bnewbold In my opinion since it's about Research Papers/Journals majorly, people visiting would be acquainted with the style. Hence, the firsts one 'Vol. 1, Iss. 2, p3-4' seems legit.

AniketShahane commented 3 years ago

@bnewbold I'd like to help with this issue. I'm new to the open source contribution process. I'm fairly skilled with python and have worked with fastAPI quite a bit. I'd like to start contributing to this project. Can you guide me? What should be my first steps? Thanks and Regards.

bnewbold commented 3 years ago

I would start by writing tests, or modifying existing tests. I don't actually know if there is a good way to do tests of jinja2 template "macro" functions, but you could probably figure something out by calling the jinja2 template macros directly.

The place this code path would live is in the "journal row" macro: https://github.com/internetarchive/fatcat-scholar/blob/master/fatcat_scholar/templates/search_macros.html#L141

Implementation needs to handle:

While the basic feature is pretty simple, I think that figuring out how to do testing, and then handling translations, could make it a bit tricky.