Adaption to Yii 3.x planned?

2amigos / yii2-usuario

Highly customizable and extensible user management, authentication, and authorization Yii2 extension

https://github.com/2amigos/yii2-usuario

Other

294 stars 142 forks source link

Adaption to Yii 3.x planned? #298

Closed schmunk42 closed 5 years ago

schmunk42 commented 5 years ago

Are you planning to update this module to Yii 3.x? I know it's pre-alpha yet, but this is more a general question...

maxxer commented 5 years ago

Of course we want to keep the extension up to date.

Most of the devs are currently busy, but we'll certainly make usuario yii3 compatible

tonydspaniard commented 5 years ago

I am coming back this week :)

philippfrenzel commented 5 years ago

any update on this? ;)

YiiRocks commented 5 years ago

I would say bootstrap4 first, as that is actually released already

tonydspaniard commented 5 years ago

@schmunk42 @Thoulah @philippfrenzel @maxxer What do you think about the following, split the module into the following packages:

2amigos/yii3-usuario -> core library
2amigos/yii3-usuario-b4-views -> sample bootstrap views (other packages may be created -i.e. material-views, tailwind-views, etc)
2amigos/yii3-usuario-gpdr -> package to provide gdpr mechanism
2amigos/yii3-usuario-2fa -> package to provide two factor mechanism

I think keeping this library structure it will become hard to maintain. So I am thinking to split its code. What do you guys think? Any ideas welcome.

YiiRocks commented 5 years ago

I think that's the same way Yii3 is going. Although putting views outside of main package might be a bit weird (if I understand correctly). In my own project I created a few classes inside bootstrap4 (sub)namespace. In case bootstrap5 differs I can just add classes into a new bootstrap5 namespace and support both.

schmunk42 commented 5 years ago

I'd recommend to prefix them with yii-, see https://github.com/yiisoft/docs/blob/master/000-packages.md - since we've switched to semantic versioning.

tonydspaniard commented 5 years ago

@Thoulah the reason why views are outside is because I am planning to create a core version that could be used via api or desktop (web controllers). So, there would be an API package and a WEB package that will include controllers and a reference to the view they would like to use. Thats the idea.

Edit: I also believe that RBAC should be optional on this package also. I would split it as much as i can.

YiiRocks commented 5 years ago

Now it makes sense :+1:

tonydspaniard commented 5 years ago

I have been talking to Alex Makarov and he told me that migrating to Yii3 still too early. Maybe we should wait a little.

tonydspaniard commented 5 years ago

re #323