ApplicationDecorator class

[dmitrypol]

A common pattern in Rails is to have ApplicationController, ApplicationMailer, etc. The class inherits from base and developer generated code inherits from it. This provides a useful place to implement common functionality.

We are manually introducing this in our app it but it could be a built-in feature. Does anyone have ideas / suggestions?

class ApplicationDecorator < Draper::Decorator
  delegate_all
  ...
end
class UserDecorator < ApplicationDecorator
  ...
end

[anbiniyar]

I did the same thing as well. +1

[Mehonoshin]

Nice idea!

[francirp]

👍 I do the same thing. I typically implement currency and percent class methods in my BaseDecorator. If you're curious I wrote a post on it here: https://medium.com/@ryan_p_francis/how-decorators-saved-my-rails-app-c5cc18ef7d2e

[akodkod]

Yeap, good idea! We could create rake task rake draper:install to create this file automatically.

[francirp]

Yeah exactly. Happy to take a first punch at a PR for this if it's something the team wants to pursue. Who makes decisions like this on the Draper team? @seanlinsley what are your thoughts on the subject?

[seanlinsley]

👍 sounds like a good idea

[codebycliff]

This seems like a good idea. Is anyone still interested in taking a stab at a PR?

[chuck-john]

@codebycliff, this is a great idea. I'll get a branch started.

[codebycliff]

This was merged in #796.