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.

Sign up for GitHub

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

Jump to bottom

Fix #2417: DOI no longer contains URL #2425

Merged

koppor merged 3 commits into master from removeURLFromDOI

Dec 22, 2016

Member

tobiasdiez commented Dec 22, 2016 •

edited

Loading

Counterproposal to #2424 😈

Change in CHANGELOG.md described
Tests created for changes
Screenshots added (for bigger UI changes)
Manually tested changed features in running JabRef
Check documentation status (Issue created for outdated help page at help.jabref.org?)
If you changed the localization: Did you run gradle localizationUpdate?


          Fix #2417: DOI no longer contains URL

97055d1

tobiasdiez added the status: ready-for-review label

Contributor

simonharrer commented Dec 22, 2016

I did not want to introduce this dependency. Less dependencies seemed better to me.

simonharrer and others added 2 commits

December 22, 2016 14:56


          Fixes tests again.

011dbea


          Merge branch 'master' into removeURLFromDOI

a44dab1

koppor merged commit c1e97f0 into master

koppor deleted the removeURLFromDOI branch

December 22, 2016 20:29

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: ready-for-review