Skip to content
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 citation info to export screen #1545

Merged
merged 25 commits into from Nov 20, 2023
Merged

Add citation info to export screen #1545

merged 25 commits into from Nov 20, 2023

Conversation

VeenDuco
Copy link
Contributor

Here a pull request to start a discussion on what adding citation information should look like. This relates to #1402 and #1403.

What I did now is to add a simple box on the export screen with the citation information. This looks like the figure below. Any thoughts?

citation_example

Copy link
Member

@J535D165 J535D165 left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! Very welcome. First brief review from my side.

@VeenDuco
Copy link
Contributor Author

The requested changes should now be implemented and Rens' additions are also included.

@george-gca
Copy link

It would be really useful to have a citation version in bibtex.

@Rensvandeschoot
Copy link
Member

I changed the text into buttons and added citations including BibTex (ping @george-gca ) . I only want to update the documentation with these screenshots.

export_screen

cite

@J535D165 J535D165 merged commit a793b06 into asreview:master Nov 20, 2023
10 checks passed
@J535D165
Copy link
Member

Thanks all!

@J535D165 J535D165 changed the title add citation info to export screen Add citation info to export screen Nov 20, 2023
cskaandorp pushed a commit to cskaandorp/asreview that referenced this pull request Apr 22, 2024
Co-authored-by: Rens van de schoot <a.g.j.vandeschoot@uu.nl>
Co-authored-by: Jonathan de Bruin <jonathandebruinos@gmail.com>
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.

None yet

4 participants