-
Notifications
You must be signed in to change notification settings - Fork 10
Loses the ability to use rails strong params #9
Comments
@joaquimadraz could you see that ? |
@alexandru-calinoiu still an issue to you ? let me know ! cheers |
@tdantas Sorry for the delay. I haven't check this issue. Next couple of weeks will be busy for me. Can you check that? |
take your time @joaquimadraz. |
Hey @alexandru-calinoiu! Sorry, I need some context to help you on this one. Thanks and sorry for the late response! |
The idea is I want to implement some sort of a contract for my context. So if the before method for a use case I ca do:
Does this make sens? |
Most of the times, I validate the context and if something is missing I just call If something like this should exist, maybe as class method? class FindInvoice < UseCase::Base
required_context :current_user, :invoice_id
def perform
...
end
end @tdantas what do you think? |
👍 looks great ! another heads up. cheers |
With the context wrapping the
params
, require does not work anymore, how about adding therequire
in theContext
.Does it sound like a good idea? I can put a PR if you want.
The text was updated successfully, but these errors were encountered: