search

Mordhin / starships-looters

3 stars 2 forks source link

Added pundit basic setup #35

Closed JPS09 closed 3 years ago

JPS09 commented 3 years ago

I added the gem and Bundle Installed. (You may have to do it locally and in prod when this branch will be merged) I also added pundit lines in application controller. That's all.

JPS09 commented 3 years ago

Since it's a big feature, I blocked the merge feature to avoid a missclick, we will merge it when everybody reviewed it at the end

JPS09 commented 3 years ago

C'est ce que j'ai compris aussi, je ferai chaque setup après qu'une user story ait été merge sur le master en faisant un commit sur cette branche. ça sera plus propre et on pourra mieux gérer s'il y a un bug.

JPS09 commented 3 years ago

Je vais clôturer ce pull request car le git rebase ne fonctionne pas comme je l'imaginais/ comme je le souhaitais. On repartira depuis le dernier commit du master lorsqu'on décidera de se taper la config entière.