search

sciencehistory / chf-sufia

sufia-based hydra app
Other
9 stars 4 forks source link

Sharing permissions with an additional user erases first user #283

Closed catlu closed 7 years ago

catlu commented 7 years ago

capture

After I hit save, Jim's user ID disappears.

catlu commented 7 years ago

I think adding multiple in one try doesn't work either. It erases everyone but the depositor and the last person added.

hackartisan commented 7 years ago

oof

catlu commented 7 years ago

I have no idea what's going on, but I'm revisiting this and the issue is gone! Gah. I'll reopen if I see it again.

catlu commented 7 years ago

Never mind--definitely still happening but seems arbitrary as sometimes one record will save okay and the next won't. I can tell it's happening when the "Apply changes to contents?" message doesn't show up after I hit save. Going back in and removing a permissions, saving, and then resharing seems to work.

hackartisan commented 7 years ago

Here's where the "would you like to apply changes?" screen was added in: https://github.com/projecthydra/sufia/pull/2700

hackartisan commented 7 years ago

I suspect this issue is linked to #301 and therefore is similarly blocked

hackartisan commented 7 years ago

I believe this is resolved as part of sufia upgrade (pr #407). @catlu please let me know if you see any more permissions bugs