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

Add parsing of conf/privilege file #7

Closed
wants to merge 1 commit into from

Conversation

Dr-Bean
Copy link
Contributor

@Dr-Bean Dr-Bean commented Dec 18, 2016

Can't upload packages with privilege file unless it's parsed, and we need that for DSM6 users.

There's at least one other file introduced with DSM6 (resource to create shared folders), maybe others...I'll have to check. We're not using those atm.

Can't upload packages with `privilege` file unless it's parsed.
@Dr-Bean
Copy link
Contributor Author

Dr-Bean commented Dec 18, 2016

Hm. I suppose this PR isn't complete without a test and a change to models.py, along with a db migration...I'll have to come back to that.

@Dr-Bean
Copy link
Contributor Author

Dr-Bean commented Dec 20, 2016

Closing, superceded by #8.

@Dr-Bean Dr-Bean closed this Dec 20, 2016
@Dr-Bean Dr-Bean deleted the patch-1 branch December 20, 2016 18:05
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.

1 participant