Should Spree::Role be a tenanted model?

[eLafo]

I think that I am missing something because I do not get which the point is for Spree::Role to be a tenanted model

If I create a new Tenant, authorization will fail, since I need to create new roles for it. Maybe I am missing something, but I think that roles should be the same for all the tenants unless they are customizable by tenant, which they are not, right now, aren't they?

[fattymiller]

I like the idea of separating the roles based on shops as this will allow different shops to have their own roles without needing to jump through hoops to fit in to a previously defined model.

New shop setups can be a pain, but this can be easily corrected by creating a default set of Roles upon Shop creation, and then allow the shop owner to make changes at their leisure.