Upgrade Affected User Tracking

[nikz]

Initial work on upgrading affected user tracking.

Ping @MindscapeHQ/ruby-team, @armstrjare, @NZKoz, @CmdrKeen for feedback :)

[NZKoz]

Rather than this weird hash which maps properties, why not use a proc? Was:

config.affected_user_method_mapping.Email << :email_address 

Could be:

  config.affected_user do |something|
    {Email: something.email}
end

Proc could take controller, rack hash thingy, etc?

etc

[nikz]

Yeah, I'm actually leaning toward that... Because everyone is going to be
slightly different and the complexity of trying to make it magical seems to
outweigh the benefits.

On Saturday, July 19, 2014, Michael Koziarski notifications@github.com
wrote:

Rather than this weird hash which maps properties, why not use a proc? Was:

config.affected_user_method_mapping.Email << :email_address

Could be:

config.affected_user do |something|
{Email: something.email}end

Proc could take controller, rack hash thingy, etc?

etc

—
Reply to this email directly or view it on GitHub
#34 (comment)
.