This repository has been archived by the owner. It is now read-only.

2 3 stable #6

Open
wants to merge 18 commits into
from

Conversation

Projects
None yet
4 participants

azimux commented Sep 23, 2010

local_addresses doesn't exist after a "include ExceptionNotification::ConsiderLocal"

This is because self.include is defined in ExceptionNotification::ConsiderLocal::ClassMethods instead of ExceptionNotification::ConsiderLocal

azimux commented Sep 23, 2010

Hmmm... I don't know how to make it request a pull only from my 2-3-stable into rails:2-3-stable

Obviously only 1 commit should be pulled into 2-3-stable and not 17 into master

Test for consider_local and local_addresses controller class methods
added by ExceptionNotification::ConsiderLocal

albeit commented on f9865f5 Oct 11, 2010

I just installed this plugin and didn't realize I had to add:

config.gem "exception_notification"

to Rails::Initializer.run. Could this be added to the README, just so others avoid the issue?

Thanks.

jjulian commented Aug 15, 2011

This tripped me up for a few minutes too, good fix in 66dc72b.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.