Skip to content
This repository has been archived by the owner on Nov 9, 2022. It is now read-only.

FIX #514 #545

Merged
merged 4 commits into from
Feb 26, 2016
Merged

FIX #514 #545

merged 4 commits into from
Feb 26, 2016

Conversation

jmeekhof
Copy link
Contributor

@jmeekhof jmeekhof commented Oct 30, 2015

Update to allow non-admin users to deploy. #514

Update to allow non-admin users to deploy.
@grtjn
Copy link
Contributor

grtjn commented Nov 12, 2015

I think this might need to be piled on top of #518, but not entirely sure. ERU was introduced there, didn't exist before yet..

@jmeekhof
Copy link
Contributor Author

@grtjn You are correct. It's related to the same issue. I believe the #518 fixed a deploy for REST extensions, and this is module / content deploy issue
.

@jmeekhof
Copy link
Contributor Author

has this been merged, but the Pull Request left open?

@dmcassel
Copy link
Collaborator

This one hasn't been merged yet, but I just looked and the only non-whitespace change is adding update permission in one spot. I'll merge it.

dmcassel added a commit that referenced this pull request Feb 26, 2016
@dmcassel dmcassel merged commit 9e0c693 into marklogic-community:dev Feb 26, 2016
@jmeekhof
Copy link
Contributor Author

Cool, thanks. BTW, the white space changes all follow the .editorconfig settings, so that's a good thing too.

@jmeekhof jmeekhof deleted the src_perms branch February 26, 2016 20:55
@dmcassel
Copy link
Collaborator

Yes, I was happy to see those.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants