Skip to content

Add support Mass-Assignment Role of ActiveRecord + specs #657

Closed
wants to merge 4 commits into from

4 participants

@rvanlieshout

Finished work on pull request #577 by fixing code and adding specs for an adapter called active_record_31. ActiveRecord 3.0 is still supported using this fork.

@martijn
martijn commented Jun 19, 2012

For clarity's sake I would suggest renaming the :assignment parameter to something like :assign_as

@andhapp andhapp added a commit that referenced this pull request Jun 23, 2012
@andhapp andhapp Fix for pull request #657. 30db2eb
@andhapp andhapp added a commit that referenced this pull request Jul 12, 2012
@andhapp andhapp Fix for pull request #657. ff670ed
@andhapp andhapp referenced this pull request Jul 12, 2012
Open

Fix pull request 657 #686

@andhapp
Collaborator
andhapp commented Jul 12, 2012

Fixed the pull request so that it merges cleanly. Here's the new pull request.

Closing this one.

@andhapp andhapp closed this Jul 12, 2012
@ghost Unknown added a commit to bignerdranch/cancan that referenced this pull request Apr 16, 2013
@andhapp andhapp Fix for pull request #657. 6e0ef16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.