developeruz / yii2-db-rbac

Dynamic control of access rights in YII2
MIT License
110 stars 33 forks source link

Доступ к модулю по ролям #18

Closed Pashkinz92 closed 7 years ago

Pashkinz92 commented 9 years ago

Думаю, будет удобно, когда для доступа к модулю будут использоваться уже созданные роли

developeruz commented 7 years ago

Дело в том, что я умышленно не хочу вводить роли по умолчанию в модуль. Чем заставлять каждого разработчика модулей поддерживать какие-то роли, проще чтобы ни в каких модулях небыло прописанного AccessControll и правами для всех модулей занималось одно единтственное поведение (например мое из этого-же пакета).

Ваши правки мне нравятся в первую очередь потому, что без ролей совсем нельзя, например когда роли хранятся в файлах и поведение не используется, то хорошо иметь возможность хоть как-то ограничить доступ к модулю. Исправьте пожалуйста конфликты и я приму пул.

Pashkinz92 commented 7 years ago

Огого, давно это было. Тааакс.... я чет не до конца понял, каким образом те конфликты можно исправить то....может быть подскажете, куда смотреть то надо ? )

developeruz commented 7 years ago

Нужно сделать git pull upstream master и git push. Я думаю конфликты сами смержатся. Просто у меня нет доступа к вашей ветке.

Pashkinz92 commented 7 years ago

Похоже и у меня не получится сделать, fatal: 'upstream' does not appear to be a git repository fatal: Could not read from remote repository.

Please make sure you have the correct access rights and the repository exists.

developeruz commented 7 years ago

Сделайте сперва:

git remote add upstream git://github.com:developeruz/yii2-db-rbac.git

Затем

git pull upstream

И если конфликтов нет - то

git push

PS: если не получится, то дайте знать, скопирую изменения в файлах и запушу сама. Просто хотелось бы ваше авторство сохранить, поэтому будем пытаться через пул-реквест.

Pashkinz92 commented 7 years ago

Что бы уже не парится, поделал через интерфейс gihub

developeruz commented 7 years ago

Из-за конфилктов смержить ваши изменения не возможно. Я сделала своим коммитом, включив вас в changelog. Этот пул реквест закрываю.

Pashkinz92 commented 7 years ago

Ok :-)