On Fri, 2011-04-22 at 14:01 +0200, Tomas Sedovic wrote:
On 04/20/2011 11:26 PM, Jason Guiditta wrote:
On Tue, 2011-04-19 at 16:47 +0200, tsedovic@redhat.com wrote:
From: Tomas Sedovictsedovic@redhat.com
Thanks for looking at the patch Jay.
I must say I'm surprised by your experience however. The patch does include an update to the seeds.rb and the migrations looked and seemed to work right for me.
What do you mean by running the migrations conditionally? The code in this patch leaves any records that don't have the old values untouched. I tried to look at other migrations we have to see if there's something else I should have done but I didn't find anything.
Anyway, I tested the whole thing again to try and replicate your behaviour but everything worked as it should.
Tomas, I am unable to replicate the behavior I saw either, so I think we can ignore it. You are right on the other points, not sure what I was thinking there, so ACK.
-j