Petrik de Heus

Results 175 comments of Petrik de Heus

The [API docs](https://edgeapi.rubyonrails.org/classes/ActiveRecord/DelegatedType.html) also use the pseudo-quotes. > But there’s a third way: Delegated types. With this approach, the “superclass” is a concrete class that is represented by its own...

You could try explicitly unsetting the content attribute: ```ruby if node.key? "content" node["content"] = sanitize_content_attachment(node["content"]) end # Add this line to unset it: node.delete("content") if node["content"].blank? ``` https://github.com/rails/rails/blob/52c21f90669ce59adf7deef85a44b6b1a46928fa/actiontext/lib/action_text/content.rb#L100-L102

Thanks for the PR @tomrossi7 ! This seems very similar to preprocessed variants: https://github.com/rails/rails/pull/47473

Maybe we should have non boolean values for preprocessed? `preprocessed: :later` `preprocessed: :immediate` Or maybe :immediate should be the default?

Rails main is already 8.1.0-alpha since october: https://github.com/rails/rails/blob/main/RAILS_VERSION

@jarthod Sorry for the late reply. To get this backported, open a new issue based on your comment: https://github.com/rails/rails/pull/52094#issuecomment-2654282368 .

I"m not sure we should be documenting this. There are a lot more configuration settings that have been added/removed from environment files. Unlike the framework defaults that we do document...

Seeing there are [previous PRs,](https://github.com/rails/rails/pull/51845) these PRs have precendence over this PR. Unless they don't have any activity.

I think these are only set in sprockets and propshaft. If you use neither the setting arent' present. I could add that to the `config.assets` opening paragraph.

Hmm, `config.assets.css_compressor` is [set by sassc-rails](https://github.com/sass/sassc-rails/blob/master/lib/sassc/rails/railtie.rb#L74) which hasn't been updated in 5 years.