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

Fix Paranoia to work with validates_uniquness_of (rails4 branch) #121

Closed
wants to merge 2 commits into from

Conversation

codeodor
Copy link
Contributor

For the rails4 branch, in pursuit of #114

@codeodor
Copy link
Contributor Author

@radar just wondering if you had a chance to look at this PR and the one in #121 too. I got bit by this again, so it reminded me of these. 😄

@ethangunderson
Copy link

Any update on this one? I never learn, so this keeps biting us.

@radar
Copy link
Collaborator

radar commented Jan 26, 2015

If there was an update you can be sure it would be posted here. There is no secret place where things about this gem get discussed. It's all done here out in the open.

On 27 Jan 2015, at 09:59, Ethan Gunderson [email protected] wrote:

Any update on this one? I never learn, so this keeps biting us.


Reply to this email directly or view it on GitHub.

@ethangunderson
Copy link

I apologize, that was a poorly worded comment on my part. Let me try again. I would like to see this merged, is there anything I can do to help it along?

@codeodor codeodor changed the title Failing test that shows Paranoia not working with validates_uniquness_of (rails4 branch) Fixed & Ready to merge: Failing test that shows Paranoia not working with validates_uniquness_of (rails4 branch) Jun 18, 2015
@codeodor
Copy link
Contributor Author

This one should be ready now too.

@codeodor codeodor changed the title Fixed & Ready to merge: Failing test that shows Paranoia not working with validates_uniquness_of (rails4 branch) Fix Paranoia to work with validates_uniquness_of (rails4 branch) Jun 18, 2015
@codeodor
Copy link
Contributor Author

This will also fix #175

@radar radar closed this Jun 20, 2015
radar added a commit that referenced this pull request Jun 22, 2015
Fix for #122, #121 and #175 on Rails 4 branch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants