Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Fix #1062 Merge entry with DOI information now also applies changes to entry type #1064

Merged
merged 1 commit into from
Mar 30, 2016
Merged

Fix #1062 Merge entry with DOI information now also applies changes to entry type #1064

merged 1 commit into from
Mar 30, 2016

Conversation

stefan-kolb
Copy link
Member

<describe the changes you have made here: what, why, ...>

  • Change in CHANGELOG.md described
  • Tests created for changes
  • Screenshots added (for bigger UI changes)

@stefan-kolb stefan-kolb added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Mar 30, 2016
@koppor
Copy link
Member

koppor commented Mar 30, 2016

No test, because a test requires an online connection? 😇
Other than that: LGTM 👍

@tobiasdiez
Copy link
Member

LGTM 👍

@koppor koppor merged commit 9be400a into master Mar 30, 2016
@stefan-kolb stefan-kolb deleted the merge-entry-type-change branch March 30, 2016 17:58
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@stefan-kolb @koppor @tobiasdiez