Fix premature db access #407

Closed
wants to merge 1 commit into
from

Projects

None yet

3 participants

@betamatt
Collaborator

Seems to solve premature database access described in #405 without too much hackery.

Someone should review to see if I'm missing something this is going to kill.

@luismreis

hey @collectiveidea ( ? ), can you take a look at this ?

@luismreis

@betamatt, can I be the reviewer ? What do you need ? If it's just a "makes or breaks" I can test it. The code seems fine (I don't know railties so well).

@albus522
Member
albus522 commented Nov 5, 2012

I would suggest delaying only the active record extensions and not the entire extension. This does appear to be a good solution, I would simply like it to be a little more specific. Do you want to make the updates, so that you still get credit?

@albus522
Member

No longer needed

@albus522 albus522 closed this Sep 24, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment