search

OregonDigital / hyrax-migrator

A Rails engine to facilitate migrating works into a Hyrax based application
MIT License
0 stars 0 forks source link

Adds in actor which utilizes service to add permission to collection #357

Open straleyb opened 1 year ago

straleyb commented 1 year ago

@lsat12357 I want to write some tests for this, but I'd like your input on this. I plugged into the actor stack to utilize a new actor which adds the permissions. The service already exists in hyrax, so i'm just utilizing that here instead of creating my own Hyrax::Collections::PermissionsCreateService. It still utilizes the state machine structure and adheres to that. Only difference is that i'm calling the method on the service in a different spot than the other ones typically do (seen on lines 48 and 53)

straleyb commented 1 year ago

I meant to draft this. Not ready for merge yet since it needs tests.