Rafael Mendonça França comments

Results 515 comments of


                                            Rafael Mendonça França

Include ActionMailer::DeliverLater::Mixin in ActionMailer::Base by default

One thin we need to do is to fix this anonymous module to call `super`.

Better Rails defaults

Both ideas are great, but I was wondering, why not `app/assets/sprites` instead of `app/sprites`?

Should this be a first citizen of ActiveRecord?

You mean like `saved_changes` added in rails 5.1?

Should this be a first citizen of ActiveRecord?

Oh, I see now. Yeah, it seems fine to be as a new method, but I don't think we should change an existing behavior. @sgrif may have a good option...

Add prefix to attribute definition

LGTM

typed_store attributes is marked as changed when inspecting instance

Try changing `biding.pry` with `user.inspect`

typed_store attributes is marked as changed when inspecting instance

Yes

super: no superclass method `extract_default'

Do you have the entire backtrace? That can help us to find where that method is being called.

super: no superclass method `extract_default'

Ah, I see what is happening. Since this gem don't support Rails 6 yet, bundle may had downgraded you to an incompatible version. `ActiveRecord::TypedStore::Column` don't exist anymore in the current...