Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reduced column sizes in taggings table migration. #187

Closed
wants to merge 16 commits into from

Conversation

HuckyDucky
Copy link
Contributor

... facilitate index creation on MySQL databases with a default storage engine of MyISAM.

… to facilitate index creation on MySQL databases with a default storage engine of MyISAM.
@travisbot
Copy link

This pull request fails (merged d00214f into 69cfd33).

@travisbot
Copy link

This pull request passes (merged 80210b3 into 69cfd33).

@travisbot
Copy link

This pull request fails (merged 9e44469 into 69cfd33).

@travisbot
Copy link

This pull request fails (merged 0757c04 into 69cfd33).

@travisbot
Copy link

This pull request fails (merged 36a5036 into 69cfd33).

@travisbot
Copy link

This pull request fails (merged 054f3df into 69cfd33).

@travisbot
Copy link

This pull request fails (merged 94209aa into 69cfd33).

@travisbot
Copy link

This pull request fails (merged 2ba9422 into 69cfd33).

@travisbot
Copy link

This pull request fails (merged ce23bc7 into 69cfd33).

@travisbot
Copy link

This pull request fails (merged 3d1d281 into 69cfd33).

@steveyken
Copy link
Member

Thanks for this. It's heading in the right direction. We also need to add support for custom fields whitelisting too. I believe setting config.active_record.whitelist_attributes = false for now should work around the issue but I wholeheartedly agree with actually fixing the problem rather than working around it!

@steveyken
Copy link
Member

Thanks for your work on this. I've added commit d00214f to master. This will ensure mysql / MyISAM setups will migrate from scratch. Will leave attr_accessible for another day so leaving this pull request open.

@steveyken
Copy link
Member

I've pulled the attr_accessible commits out into a new branch called 'attr_accessible' in FFCRM. I'll close this pull-request now and we can focus on that branch for the remaining attr_accessible features.

@steveyken steveyken closed this Jan 30, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants