ExLibrisGroup / alma-csv-user-load

Load users to Ex Libris Alma from CSV files
2 stars 0 forks source link

Add Primary ID for failed deletes? #13

Closed bkelm closed 3 years ago

bkelm commented 4 years ago

This works great for deletes, I know we get Primary ID's of processed results, would it be possible to get Primary ID's for failed ones? I know I could go back to Analytics and create another report the next day to gather too.

"Failed: Request failed: User cannot be deleted - A fine fee of 29.00 USD is attached to this user."

jweisman commented 3 years ago

Thanks for your request. We'll look into adding this.

jweisman commented 3 years ago

Hi @bkelm . We added the primary ID, first name, and last name to the error message when a record can't be added/updated. Hope this helps.

Failed: Given email type (school) is not supported for given user record type (STAFF). (csvload102, Weisman, Hank)
bkelm commented 3 years ago

Close, but I need something about the user when it can not be deleted.

jweisman commented 3 years ago

Hi @bkelm - can you elaborate? What additional information would you like to see other than the primary ID, first name, and last name?

bkelm commented 3 years ago

I think just the primary ID is fine, as the user information is still in Alma.

jweisman commented 3 years ago

Hi @bkelm - maybe we're not on the same page : ) The primary ID should appear now in the error when a record cannot be deleted. The log will show the error message, and then the primary ID, first name and last name in parentheses, like this:

Failed: Given email type (school) is not supported for given user record type (STAFF). (csvload102, Weisman, Hank)

Are you able to see that when you get a failed delete?

bkelm commented 3 years ago

I am not able to see it, on version 1.6.1. I get the following in the log:

Failed: Request failed: User cannot be deleted - User is linked to an invoice. Failed: Request failed: User cannot be deleted - A fine fee of 5.25 USD is attached to this user. Failed: Request failed: User cannot be deleted - A fine fee of 36.50 USD is attached to this user. Failed: Request failed: User cannot be deleted - User is linked to a loan

jweisman commented 3 years ago

Hi @bkelm - you’re right!! Looks like I completely missed that it wasn’t showing the primary Id for failed deleted.

We fixed this in the current release. Can you give it a try end let us know if it works for you?

bkelm commented 3 years ago

Works perfect now.

Failed: Request failed: User cannot be deleted - User is linked to a loan (**, *, )

Also verified to me that the Apps are always updated without having to re-install them.

jweisman commented 3 years ago

Great! Thanks for explaining patiently what I was missing : )