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

DS-2779 #4

Merged
merged 2 commits into from Oct 20, 2015
Merged

DS-2779 #4

merged 2 commits into from Oct 20, 2015

Conversation

pnbecker
Copy link

Hi Kevin,

I rebased your PR and added some more fixes to it. Please merge it to your PR before merging it to DSpace.

Regards,
Pascal

@KevinVdV
Copy link
Owner

This is an unusual amount of commits, hard to see what has changed exactly & I'm seeing that the pull request cannot be merged yet.

@pnbecker
Copy link
Author

Sorry, this was caused by the rebase against DSpace/master. Here you get a clean PR.

KevinVdV added a commit that referenced this pull request Oct 20, 2015
These fixes make sense, merging, thanks for the help @pnbecker.
@KevinVdV KevinVdV merged commit 4e60beb into KevinVdV:DS-2779-fix-versioning-xmlui Oct 20, 2015
samuelcambien pushed a commit to samuelcambien/DSpace that referenced this pull request Dec 7, 2016
samuelcambien pushed a commit to samuelcambien/DSpace that referenced this pull request Dec 7, 2016
[DS-1814] remove deprecated ConfigurationManager in favour of Configu…
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants