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

tasks: background fixity checker #29

Merged
merged 2 commits into from
Mar 3, 2016
Merged

Conversation

lnielsen
Copy link
Member

No description provided.

* Adds linking between location, buckets, objects and file instances
  to viewing easier.

Signed-off-by: Lars Holm Nielsen <lars.holm.nielsen@cern.ch>
@lnielsen
Copy link
Member Author

Depends on inveniosoftware/invenio-admin#17

@lnielsen lnielsen force-pushed the fixity branch 3 times, most recently from 0b41a79 to 01f12aa Compare March 2, 2016 07:32
@lnielsen lnielsen assigned drjova and unassigned lnielsen Mar 2, 2016
@drjova
Copy link
Member

drjova commented Mar 2, 2016

@lnielsen from models serialize is missing. Are we gonna use the serializers from #28?

# -*- coding: utf-8 -*-
#
# This file is part of Invenio.
# Copyright (C) 2015, 2016 CERN.
Copy link
Member

Choose a reason for hiding this comment

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

only 2016?

* Adds background fixity checker with support for starting it from the
  administration interface. (closes inveniosoftware#11)

Signed-off-by: Lars Holm Nielsen <lars.holm.nielsen@cern.ch>
@lnielsen
Copy link
Member Author

lnielsen commented Mar 2, 2016

@drjova Serialisers will be rewritten using the same logic as what has just been implemented in Records-REST.

@drjova
Copy link
Member

drjova commented Mar 2, 2016

@lnielsen ok! the only problem is that the example's API is broken :/ Is it possible to add dummy serialisers for now WDYT? Otherwise is good to go :)

@lnielsen
Copy link
Member Author

lnielsen commented Mar 3, 2016

@drjova Yes, #28 is adding some dummy serialisers so we can merge that one right after.

@drjova
Copy link
Member

drjova commented Mar 3, 2016

:shipit:

@drjova drjova merged commit 328175f into inveniosoftware:master Mar 3, 2016
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.

None yet

2 participants