nondestructive_migrations icon indicating copy to clipboard operation
nondestructive_migrations copied to clipboard

make the data_migration table name work with table_name_prefix and ta…

Open polarlights opened this issue 8 years ago • 2 comments

I know the gem from a friend. I read the source code and I see the code below:

#lib/active_record/data_migration.rb#L12
def table_name
    "#{table_name_prefix}data_migrations#{table_name_suffix}"
end

However, the schema file under the templates folder, the table name is fixed. I think it will be better if we can support the table_name_prefix and table_name_suffix as your wrote in the code.

It's not a bug, just to make it better. :)

polarlights avatar May 07 '16 13:05 polarlights

Thank you !

I will pull this against a few of my apps, configure & run it, and then merge this in assuming everything looks good. Please give me a few days. I will be sure to put this on my TOOD list

jasonfb avatar May 25 '16 11:05 jasonfb

So sorry I don't know why I didn't see this 3 weeks ago when you submitted it.

jasonfb avatar May 25 '16 11:05 jasonfb