-
Notifications
You must be signed in to change notification settings - Fork 15
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
Add support for scite.ai #375
Comments
Hi ! |
It is available (as you see in the status). Sorry for the late answer. I assume, you already worked on something else. |
Hello, I'm a university student with an assignment to contribute to an open source repo, may I have a go at working on this issue? |
@taliashark77 OK, I assign you. |
@taliashark77 Please be advised that this a difficult issue. Guidance:
For the UI, two options A. Put on top in "Citation relations" tab (and rename to the tab to "Citation information" B has the advantage that the APIs fetching the data are only called if the user wants it. It has the disadvantage that the user cannot get a complete overview of the "Citation information". Nevertheless, go for option B and a separate tab. That will be more simpler to implement and test. |
As a general advice for newcomers: check out Contributing for a start. Also, guidelines for setting up a local workspace is worth having a look at. Feel free to ask here at GitHub, if you have any issue related questions. If you have questions about how to setup your workspace use JabRef's Gitter chat. Try to open a (draft) pull-request early on, so that people can see you are working on the issue and so that they can see the direction the pull request is heading towards. This way, you will likely receive valuable feedback. |
@koppor thanks for assigning me however I believe I do not actually know enough about APIs to work on this issue, is it ok if I'm unassigned and find another issue to work on for my assignment? Thanks |
Hey @koppor, because taliashark77 is no longer working on this issue can be assigned to it. Thanks |
@u7500248 Done. Looking forward! |
* task: first cut of Scite tab and associated preference * task: comments and minor refactoring * task: Updated readme.md for issue #375 * fix: Import ordering and markdown space * fix: imports and whitespace (checkstyle) * fix: imports and whitespace (checkstyle) * fix: imports and whitespace (checkstyle) * fix: Removed unnecessary parentheses (OpenRewrite) * fix: Unit tests and localisation keys * fix: Removed unnecessary heading in CHANGELOG.md * fix: Made SciteTallyDTO a record type and moved it to new file * fix: Made the scite.ai base url a constant * fix: Exception handling now catches most specific exceptions, and rethrows them as FetcherException * fix: style for scite message box * fix: Catch specific exception and display error message rather than throwing a generic unchecked exception * fix: Localization for SciteTab error messages, and "Scite" name is no longer localized * some small refactoring * checkstyle and rename * Update CHANGELOG.md * Update EntryEditorTab.java * move and checkstyle * fix l10n and rename --------- Co-authored-by: Siedlerchr <siedlerkiller@gmail.com> Co-authored-by: Carl Christian Snethlage <50491877+calixtus@users.noreply.github.com>
https://scite.ai/
The text was updated successfully, but these errors were encountered: