Pablo Brasero

Results 297 comments of Pablo Brasero

Your proposal sounds good to me. Go for it! And thank you for looking into this :slightly_smiling_face:

As mentioned in comments above, I have created a branch that uses your code and I'm experimenting with it. It's at https://github.com/thoughtbot/administrate/compare/main...pablobm:use-cases-for-contextualize. My goals are: - Work with your proposal...

I'm not sure about this one. I think that Administrate should be independent from authentication, and users should implement their own. If anything, we should make sure that this is...

Thank you for letting us know of this use case. We'll look into it. Ultimately, we want Administrate to be able to detect all these cases correctly so that we...

Yeah, we may need something like that. Ideally I was thinking we could have it in `Administrate::ApplicationController`, so that users don't have to set it up. However if we just...

Thank you for this @nhippenmeyer! Eventually I think we should deprecate `policy_scope_admin`, but until then this is a bug that should be addressed. Would you be able to make the...

Another option, I think, would be for each look to be a custom field type such that `to_partial_path` pointed to the correct partial in each case. The downside is that...

@alex-benoit, thank you for your report. At the moment, the titles are defined in templates (index, show, new, edit), so I think @jubilee2's suggestion is the way to go. Having...

That's a valid point. I had a look and my current impression is that we may want to change the `belongs_to` templates to render the value even if it's not...

Do you want to prepare a PR to prototype your proposal? No need for specs or docs just yet. Only to see what it would look like and discuss.