-
Notifications
You must be signed in to change notification settings - Fork 0
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
Implement ShortScience integration #3
Comments
It seems we might be able to access ShortScience by arxiv ID through the URL pattern |
It doesn't seem to be quite as easy with DOI numbers. Maybe a good initial solution would be to search by arXiv ID and perhaps title. |
Actually, one way we could implement this without having to deal with web scraping would be to simply provide the user with a link to a search of ShortScience with the article's title as the query string. What do you guys think? |
That could be a solution, do you have any initial thoughts on where in the GUI such a link could be placed? One thought that I have is to place it in one of the tabs when clicking on an entry (i.e. in the entry editor). But it also feels a bit wasteful in regards to space to just put a link there. |
|
I'll write a mockup for this |
This commit adds a simple integration of ShortScience.org in the form of an additional context menu action when an entry is selected. This action opens the user's web browser to the search page of ShortScience with the title of the paper entered. Essentially, this is our MVP for the integration towards ShortScience bar some testing. [Issue: #3]
I think this is a great place to integrate the funcitonlaity! |
I have a mockup up and running on branch issue/3 :) |
After a lot of thorough experimentation, I think the best solution for this would be to redirect the user to the search page as is currently implemented on branch issue/3. I experimented with parsing and creating direct links to the article, but ShortScience is too inconsistent with their format for it to be worth our effort. I believe that this feature is essentially complete now. |
Resolved, and being tracked through PR review on JabRef instead. |
We need to integrate ShortScience into JabRef.
The text was updated successfully, but these errors were encountered: