Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Fixing issue #534 (now it works on Rails 2.3.8) #538

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
Contributor

aya-soft commented Jul 27, 2011

No description provided.

Owner

mike-burns commented Jul 28, 2011

Hi @aya-soft, is it possible to write a test for this?

Thanks,
-Mike

Contributor

sikachu commented Jul 29, 2011

@mike-burns this one doesn't need the test yet because we don't have an integration test which can test this. I'm working on it.

Contributor

sikachu commented Jul 29, 2011

However, I think the best way is to stick it under respond_to?(:set_callback) condition. So I'm amending your commit.
Thanks for sending in.

@sikachu sikachu closed this in 593acb1 Jul 29, 2011

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