Active Admin - Pundit #378
Conversation
ldlsegovia
force-pushed
the
pundit-admin
branch
2 times, most recently
from
9c7ceeb to
3a7a0dc
Compare
| @@ -0,0 +1,2 @@ | |||
| class BackOffice::AdminUserPolicy < BackOffice::DefaultPolicy | |||
There was a problem hiding this comment.
no sé si tiene sentido, pero deberíamos poner estas policies en una carpeta back_office?
| destination_path = [model_namespace, policy_name].compact.join("/") | ||
| template( | ||
| "../assets/active_admin/policies/#{policy_name}.rb", | ||
| "app/policies/back_office/#{destination_path}.rb" |
There was a problem hiding this comment.
ah en el proyecto quedan ahí 👌🏻 , retiro lo dicho jaja
| copy_policy("default_policy") | ||
| copy_policy("admin_user_policy") | ||
| copy_policy("page_policy", "active_admin") | ||
| copy_policy("comment_policy", "active_admin") |
There was a problem hiding this comment.
la estructura entonces quedaría por ejemplo:
- app/policies/back_office/default_policy.rb
- app/policies/back_office/admin_user_policy.rb
- app/policies/back_office/active_admin/page_policy.rb
- app/policies/back_office/active_admin/comment_policy.rb
y eventualmente: - app/policies/back_office/my_cool_model_policy.rb
eso o no?
There was a problem hiding this comment.
se ve bacán! te dejé una duda pa confirmar, pero fuera de eso, no sé si valdrá la pena hacer un comentario en el readme (de potassium o del proyecto) dado que esto no es tan común o no?
Pensé que la estructura lo hacía más obvio pero quizás no. Voy a agregarle info al README 👍🏻
There was a problem hiding this comment.
las debajo de active_admin como que me costaba procesarlas
ldlsegovia
force-pushed
the
pundit-admin
branch
2 times, most recently
from
77d22f0 to
76b1a2c
Compare
| @@ -153,6 +153,26 @@ readme: | |||
| ``` | |||
| - You can also use **any** vue bindings such as `v-for` , `:key` etc. | |||
| <% end %> | |||
| <% if get(:authorization) %> | |||
There was a problem hiding this comment.
@rjherrera le agregué el README
lib/potassium/assets/README.yml
Outdated
| `app/policies/back_office/client_policy.rb`: | ||
|
|
||
| ```ruby | ||
| class BackOffice::CLientPolicy < BackOffice::DefaultPolicy |
ldlsegovia
force-pushed
the
pundit-admin
branch
from
76b1a2c to
af40294
Compare
Different pundit settings for app and Active Admin.
Active admin policies now go inside the
BackOfficenamespace.Active admin policies now inherit from
BackOffice::DefaultPolicy.