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

Issue #5223 #5266

Merged
merged 6 commits into from
Sep 9, 2019
Merged

Issue #5223 #5266

merged 6 commits into from
Sep 9, 2019

Conversation

mmihuu
Copy link
Contributor

@mmihuu mmihuu commented Sep 1, 2019

issue #5223
Add @software and @dataset entrytype as an option in the Biblatex and Bibtex. Add fields eventitle, eventdate and venue to @Unpublished entry type. I used the MISC set of fields to supplement Software and DataSet, couldn't find documentation about specific fields that should be included in @software and @dataset entry type.
entrytype
entryTypePreferences


  • Change in CHANGELOG.md described
  • Tests created for changes
  • Manually tested changed features in running JabRef
  • Screenshots added in PR description (for bigger UI changes)
  • Ensured that the git commit message is a good one
  • Check documentation status (Issue created for outdated help page at help.jabref.org?)

@Siedlerchr
Copy link
Member

Looks good so far. Please add a changelog entry
Regarding the fields: You can look up in the biblatex documentation: 2.1.1:
https://www.ctan.org/pkg/biblatex

For the next time please do not enter the issue number in the title of the PR, because github can't link it then. If you write Closes #xxxx or Fixes #xxx then the issue will be automatically closed when the PR is merged.

@@ -177,8 +177,22 @@
.withImportantFields(StandardField.MONTH, StandardField.YEAR)
.build();


private static final BibEntryType SOFTWARE = new BibEntryTypeBuilder()
Copy link
Member

Choose a reason for hiding this comment

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

I think software and dataset are specific to biblatex and thus shouldn't be added here for the bibtex entry types.

Copy link
Member

@tobiasdiez tobiasdiez 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 quick follow-up! Looks good to me now.

@Siedlerchr Siedlerchr merged commit 02d84e1 into JabRef:master Sep 9, 2019
@khatchad
Copy link

Which biblatex styles support @dataset?

@mlep
Copy link
Contributor

mlep commented Feb 28, 2020

biblatex-bath does (https://ctan.org/pkg/biblatex-bath)

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.

5 participants