k-int / gokb-phase1

Original GOKb repo - Moving to https://github.com/openlibraryenvironment/gokb
http://www.gokb.org
Other
11 stars 5 forks source link

Ingest: Update method attribute #450

Closed kristenwilson closed 8 years ago

kristenwilson commented 9 years ago

Create an attribute that identifies the source of the last update to the component -- Refine, TSV import, manual

jhsolomon commented 8 years ago

@ianibo Can you tell me where this is located in the package record? For some reason, I don't see it...

ianibo commented 8 years ago

image

jhsolomon commented 8 years ago

thank you!

jhsolomon commented 8 years ago

The update method is not auto-populating, but it will allow me to type in the update method. Is this the correct workflow?

http://test-gokb.kuali.org/gokb/resource/show/org.gokb.cred.Package%3A291568

The last updated field did not change when I ingested the package. It should have my username, not the username of the person who created the package.

Is this related to the package not fully ingesting in Refine?

ianibo commented 8 years ago

AAAH... last updated by rather than ingest method? So there are different things - how the package was updated - refine, direct ingest, etc [Method] and person - a user - it's the person that isn't updating, right -- So we want a pseudo user called "Auto" that represents the automated ingest? Sorry for being dense -- can you confirm I've now understood?

jhsolomon commented 8 years ago

Yes, that's correct. Thanks!

On Sun, Apr 3, 2016 at 5:19 AM, Ian Ibbotson notifications@github.com wrote:

AAAH... last updated by rather than ingest method? So there are different things - how the package was updated - refine, direct ingest, etc [Method] and person - a user - it's the person that isn't updating, right -- So we want a pseudo user called "Auto" that represents the automated ingest? Sorry for being dense -- can you confirm I've now understood?

— You are receiving this because you modified the open/close state. Reply to this email directly or view it on GitHub https://github.com/k-int/gokb-phase1/issues/450#issuecomment-204923315

Jennifer Solomon GOKb Editor, Acquisitions and Discovery North Carolina State University Libraries 919-515-2743 j kristen_wilson@ncsu.eduhsolomo@ncsu.edu

jhsolomon commented 8 years ago

The user is now updating. Confirmed fix.

kristenwilson commented 8 years ago

I updated a package using Refine and the update method was not populated. I'm unsure of the status of the bulk uploader, so I have not tested that. However, I believe the point of this field is that it should be auto-populated during ingest.

jhsolomon commented 8 years ago

The Update Method field is still not populating, but the Last Updated By is working.

image

image

ianibo commented 8 years ago

Looking at http://test-gokb.kuali.org/gokb/resource/show/org.gokb.cred.Package:88993 I can see the message updated by refine... Does this look right? image

jhsolomon commented 8 years ago

Yes, that field is correct, but now the Last Updated by has the wrong username. Last updated by lamadden

On Thu, Apr 7, 2016 at 10:18 AM, Ian Ibbotson notifications@github.com wrote:

Looking at http://test-gokb.kuali.org/gokb/resource/show/org.gokb.cred.Package:88993 I can see the message updated by refine... Does this look right?

— You are receiving this because you modified the open/close state. Reply to this email directly or view it on GitHub https://github.com/k-int/gokb-phase1/issues/450#issuecomment-206927137

Jennifer Solomon GOKb Editor, Acquisitions and Discovery North Carolina State University Libraries 919-515-2743 j kristen_wilson@ncsu.eduhsolomo@ncsu.edu

ianibo commented 8 years ago

Ok - Refine processing will now additionally overwrite the last updated by field with the username of the refine user. Updating test shortly.

jhsolomon commented 8 years ago

fix confirmed

kristenwilson commented 8 years ago

fix confirmed