Santiago Bartesaghi

Hey @thewatts 👋 Regarding `#2` I agree with what you said. My opinion is that if the logic differs between the `new` and `edit` actions, then it's better to have...

@thewatts I think I'd go with this ```ruby def update @house_form = HouseForm.new(house_params.merge(id: params[:id])) if @house_form.save redirect_to @house_form, notice: "Yay!" else render :edit end end private def house_params params.require(:house).permit(:address) end...

Hey @thewatts, good point I think we should provide more help in the README regarding this 😅 What I usually do is using the `super` call with the corresponding attributes....

@thewatts Great points there, I went ahead and tried myself how I would do it in your sample app and this is the result  As you correctly mentioned, in...

@diego-suarez would you mind rebasing and merging? Thanks 🙌

Hey @georgeclaghorn did you have a chance to take a look at it? I'd like to know if this PR makes sense or if I'm missing something, thanks 😄

> Good idea! We should fix this. We also gotta find a place to document this in the Active Storage guide. @kaspth From your comments I understand that you: -...

Hey @rafaelfranca sure I can finish this but I'll need a couple of days to find some time if that's ok, thanks for the heads up

> I think linter errors should be updated in another issue @sebaherrera07 , @santib. wdyt? hmm I don't think so, I don't want to have a broken CI in the...

@robmbrooks @science I opened a PR fixing the issue