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

Add support to numericality: :other_than #1004

Closed
wants to merge 3 commits into from

Conversation

gmmcal
Copy link
Contributor

@gmmcal gmmcal commented Mar 10, 2017

Add support to validate numericality of a field when set :other_than comparison

@taschetto
Copy link

Want it :)

@mcmire mcmire added this to the v4.2.0 milestone Jun 10, 2019
@gmmcal gmmcal closed this Jul 23, 2019
@gmmcal gmmcal deleted the add_support_to_other_than branch July 23, 2019 10:54
@wayne5540
Copy link

Why did this PR get closed but not merged?

@gmmcal
Copy link
Contributor Author

gmmcal commented Mar 20, 2020

I don't remember closing this PR myself :(

I can submit a new one with same changes starting from latest code... maybe it will be better for reviewing and merging

@mcmire
Copy link
Collaborator

mcmire commented Mar 20, 2020

Ah, I just assumed you had closed it because you were tired of this PR sitting around (I don't blame you). If you still have the branch, please reopen!

@gmmcal
Copy link
Contributor Author

gmmcal commented Mar 20, 2020

I would never close for this reason 😂. I'll rebase master and do the changes again. I don't have the branch anymore, but the diff is on this PR. A lot has changed between this PR creation and current code, and I want to be closer to latest work.

@wayne5540
Copy link

lol, thanks guys, looking forward to this feature getting merged. 😂

@gmmcal
Copy link
Contributor Author

gmmcal commented Mar 21, 2020

New PR is #1282. @wayne5540 thanks for pointing up my mistake on closing this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants