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 inactive state to Object Status Property list #37

Merged
merged 1 commit into from May 19, 2016

Conversation

mjgiarlo
Copy link
Contributor

After some discussion in the Hydra community -- https://groups.google.com/d/topic/hydra-tech/2P2CCrk4LaU/discussion -- we're thinking this new state might be used as an initial state in deposit workflows where mediation must occur to "publish" an object, or could also possibly be used to "withdraw" an object without deleting it or marking it for deletion. This also achieves parity with the three object states provided by Fedora 3.

After some discussion in the Hydra community -- https://groups.google.com/d/topic/hydra-tech/2P2CCrk4LaU/discussion -- we're thinking this new state might be used as an initial state in deposit workflows where mediation must occur to "publish" an object, or could also possibly be used to "withdraw" an object without deleting it or marking it for deletion. This also achieves parity with the three object states provided by Fedora 3.
@awoods
Copy link

awoods commented May 16, 2016

Thanks, @mjgiarlo. We will discuss on the Thurs tech call:
https://wiki.duraspace.org/display/FF/2016-05-19+-+Fedora+Tech+Meeting

This seems like a constructive addition.
..and should also motivate: https://jira.duraspace.org/browse/FCREPO-1403

@ajs6f
Copy link
Contributor

ajs6f commented May 16, 2016

I'm not really against this addition right now, but note that nothing is preventing Hydra from using whatever value it wants in that position. I'd like to clean a lot of this kind of workflow-specific modeling out of this ontology and leave it to evolve in communities centered on specific workflow-implementing systems.

@awoods
Copy link

awoods commented May 16, 2016

The objState vocabulary is confined to defining exactly that, object states.
It is helpful to have a consistent, published vocabulary for these types of resource workflow properties.
@ajs6f, what would be your ideal pattern for making that happen if using objState is not ideal?

@ajs6f
Copy link
Contributor

ajs6f commented May 16, 2016

I'm saying we do not need one in the core project. There are lots of things that would be "helpful". We don't need to do all of them.

@acoburn
Copy link
Contributor

acoburn commented May 16, 2016

Wouldn't this vocab be a good candidate for fcrepo4-exts?

@ajs6f
Copy link
Contributor

ajs6f commented May 16, 2016

-exts would be better than here. The question is: into what namespace is it published?

@whikloj
Copy link
Contributor

whikloj commented May 16, 2016

The fact that anyone could publish their own ontology with any statuses they like, all they need is to say it has a type http://fedora.info/definitions/1/0/access/ResourceStatus means this doesn't need to be in the core ontology.

But I'm not really against it either.

@awoods
Copy link

awoods commented May 16, 2016

@acoburn: I was thinking the same thing.
@ajs6f: The current namespace of objState is: http://fedora.info/definitions/1/0/access/objState
..although some of the terms should be redefined under that namespace:
https://github.com/fcrepo4/ontology/blob/master/objState.rdf#L32
https://github.com/fcrepo4/ontology/blob/master/objState.rdf#L37
etc

@mjgiarlo
Copy link
Contributor Author

To clarify my intent: if we're publishing an ontology for fcrepo object states and it already includes two of the three states that came with earlier fcrepo versions, I'd sure love the third to be included also (since there are use cases for it).

I'm ambivalent re: the namespace or where this ontology lives, and will happily go along with what y'all conclude.

@awoods awoods merged commit 07c4114 into fcrepo:master May 19, 2016
@mjgiarlo mjgiarlo deleted the patch-1 branch August 10, 2016 17:43
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.

None yet

5 participants