You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Jan 28, 2021. It is now read-only.
What is the reason for SecurityModuleAppUserRegistrationServiceAbstract to have 2 abstract methods getInitialRole and getAdditionalInitialRoles?
Why not have only one method that returns Set? It will be easier to override. Otherwise, it makes you think that initial role is different that any any additional role.?
The text was updated successfully, but these errors were encountered:
Yep, I agree. Was probably just how it evolved over time.
Obviously it'll be a breaking API change, but I'm less fussy about that so long as the changelog in the README explains how to fix (perhaps worth an additional section in the README to address this).
If you want to create a PR for this, I'll be happy to merge it in.
from isisaddons-legacy/isis-module-security#46
bilgin:
What is the reason for SecurityModuleAppUserRegistrationServiceAbstract to have 2 abstract methods getInitialRole and getAdditionalInitialRoles?
Why not have only one method that returns Set? It will be easier to override. Otherwise, it makes you think that initial role is different that any any additional role.?
The text was updated successfully, but these errors were encountered: