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

global: new bucket quota validation #46

Merged
merged 1 commit into from
Apr 26, 2016

Conversation

nharraud
Copy link
Member

Signed-off-by: Nicolas Harraudeau nicolas.harraudeau@cern.ch

@nharraud nharraud self-assigned this Mar 30, 2016
@nharraud nharraud added this to the v1.0.0 milestone Mar 30, 2016
@@ -78,11 +78,11 @@
from invenio_admin import InvenioAdmin
from invenio_celery import InvenioCelery
from invenio_db import InvenioDB, db
from invenio_rest import InvenioREST
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this one should stay as is.

@lnielsen
Copy link
Member

Remember to add yourself to AUTHORS as well.

@nharraud nharraud force-pushed the quotas branch 5 times, most recently from 11c71cd to 9ba60ae Compare March 31, 2016 15:47
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 90.155% when pulling 9ba60ae on nharraud:quotas into 286cda7 on inveniosoftware:master.

@lnielsen
Copy link
Member

lnielsen commented Apr 1, 2016

LGTM. Please rebase and I'll merge.

* NEW Adds file size validation when they are added via the REST
  API (closes inveniosoftware#25).

* Fixes .editorconfig

Signed-off-by: Nicolas Harraudeau <nicolas.harraudeau@cern.ch>
@coveralls
Copy link

Coverage Status

Coverage increased (+0.6%) to 90.933% when pulling eae5edc on nharraud:quotas into 4bd76d5 on inveniosoftware:master.

@nharraud
Copy link
Member Author

nharraud commented Apr 1, 2016

@lnielsen ready. I improved one of the tests to increase coverage.

@nharraud nharraud assigned lnielsen and unassigned nharraud Apr 11, 2016
@lnielsen lnielsen merged commit eae5edc into inveniosoftware:master Apr 26, 2016
@nharraud nharraud deleted the quotas branch February 2, 2017 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

quotas: simple support
3 participants