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

Rails 5 compatibility #645

Closed
5 tasks done
tagliala opened this issue Jan 13, 2016 · 30 comments
Closed
5 tasks done

Rails 5 compatibility #645

tagliala opened this issue Jan 13, 2016 · 30 comments

Comments

@tagliala
Copy link
Contributor

tagliala commented Jan 13, 2016

  • Make all ruby tests pass
  • Make all javascript tests pass
  • Use Method#prepend instead of alias_method_chain
  • Add javascript support for case insensitive confirmation validator
  • Wait for a rack >= 2.0.0 compatible version of Sinatra
tagliala added a commit that referenced this issue Jan 13, 2016
187 runs, 191 assertions, 29 failures, 3 errors, 0 skips
Refers to #645
tagliala added a commit that referenced this issue Jan 13, 2016
187 runs, 191 assertions, 29 failures, 3 errors, 0 skips
Refers to #645
tagliala added a commit that referenced this issue Jan 13, 2016
187 runs, 191 assertions, 29 failures, 3 errors, 0 skips
Refers to #645
tagliala added a commit that referenced this issue Jan 13, 2016
Fix all ruby tests
Refers to #645
tagliala added a commit that referenced this issue Jan 13, 2016
@tagliala
Copy link
Contributor Author

It should be done, please giva a try to

gem 'client_side_validations', github: 'DavyJonesLocker/client_side_validations', branch: 'rails5'

tagliala added a commit that referenced this issue Jan 15, 2016
187 runs, 191 assertions, 29 failures, 3 errors, 0 skips
Refers to #645
tagliala added a commit that referenced this issue Jan 15, 2016
Fix all ruby tests
Refers to #645
tagliala added a commit that referenced this issue Jan 15, 2016
tagliala added a commit that referenced this issue Jan 15, 2016
187 runs, 191 assertions, 29 failures, 3 errors, 0 skips
Refers to #645
tagliala added a commit that referenced this issue Jan 15, 2016
Fix all ruby tests
Refers to #645
tagliala added a commit that referenced this issue Jan 15, 2016
@SebDE
Copy link

SebDE commented Jan 16, 2016

It worked perfekt for me - many thanks ! Just no problem with the simple-form-plugin (only small deprecated warning)

@tagliala
Copy link
Contributor Author

@SebDE 'cause the simple-form plugin still makes use of alias_method_chain, I will start a new branch there

tagliala added a commit to DavyJonesLocker/client_side_validations-simple_form that referenced this issue Jan 16, 2016
@arturatns
Copy link

Single error was found in the form submit when validating the first time and an error occurs, it is not possible to send the form again

@tagliala
Copy link
Contributor Author

does this happen only on rails5?

@SebDE
Copy link

SebDE commented Jan 21, 2016

@arturtuii can you post a example of the behaviour or the code - i just tested with my application (under Rails 5) and I'm able to submit the form every time without problems - if there are no validation errors.

If you have the problem within your project (igopizzas) - i looked at your sign-in form (there are client-side-validations used) and found the following error if i try to submit the form:

Uncaught RangeError: Maximum call stack size exceeded

The error comes from jquery and I think you have a problem with a recursive call in javascript and not with client-side-validations ...

@arturatns
Copy link

Found error now, occurred in rails.validations.custom. This working perfectly, very good this release.

@tagliala
Copy link
Contributor Author

glad to hear that 👍

tagliala added a commit to DavyJonesLocker/client_side_validations-simple_form that referenced this issue Jan 30, 2016
tagliala added a commit that referenced this issue Feb 11, 2016
Removes:
* Ruby < 2.0 code
* Rails 4 code and gemfiles
* MiniTest 4 support

Regenerate javascript
Refers to #645
tagliala added a commit that referenced this issue Feb 11, 2016
tagliala added a commit that referenced this issue Feb 11, 2016
Removes:
* Ruby < 2.0 code
* Rails 4 code and gemfiles
* MiniTest 4 support

Regenerate javascript
Refers to #645
tagliala added a commit that referenced this issue Feb 11, 2016
tagliala added a commit to DavyJonesLocker/client_side_validations-simple_form that referenced this issue Feb 11, 2016
tagliala added a commit that referenced this issue Feb 16, 2016
Removes:
* Ruby < 2.0 code
* Rails 4 code and gemfiles
* MiniTest 4 support

Regenerate javascript
Refers to #645
tagliala added a commit that referenced this issue Feb 16, 2016
tagliala added a commit that referenced this issue Feb 27, 2016
Removes:
* Ruby < 2.0 code
* Rails 4 code and gemfiles
* MiniTest 4 support

Regenerate javascript
Refers to #645
tagliala added a commit to DavyJonesLocker/client_side_validations-simple_form that referenced this issue Oct 28, 2016
tagliala added a commit to DavyJonesLocker/client_side_validations-simple_form that referenced this issue Nov 6, 2016
tagliala added a commit that referenced this issue Nov 11, 2016
Removes:
* Ruby < 2.0 code
* Rails 4 code and gemfiles
* MiniTest 4 support

Regenerate javascript
Refers to #645
tagliala added a commit that referenced this issue Nov 11, 2016
tagliala added a commit to DavyJonesLocker/client_side_validations-simple_form that referenced this issue Nov 11, 2016
@edwardmp
Copy link

Thanks for publishing this. Tried It out today but it doesn't seem to work due to the following javascript error:
rails.validations.js?body=1:188 Uncaught TypeError: Cannot read property 'remove' of undefined

@tagliala
Copy link
Contributor Author

@edwardmp please start a new issue providing all the information needed and a reduced test case: https://github.com/DavyJonesLocker/client_side_validations/blob/master/CONTRIBUTING.md#submitting-a-new-issue

tagliala added a commit that referenced this issue Dec 8, 2016
Removes:
* Ruby < 2.0 code
* Rails 4 code and gemfiles
* MiniTest 4 support

Regenerate javascript
Refers to #645
tagliala added a commit that referenced this issue Dec 8, 2016
@dreyks
Copy link

dreyks commented Dec 23, 2016

What's the status on this one?

@tagliala
Copy link
Contributor Author

@dreyks thanks for the ping.

I'm not going to release a gem in a short while, so please keep using the rails5 branch

Also, sinatra is still on beta. If we could drop the sinatra thing and run javascript unit tests against a Node.JS based server (and maybe make that work on Travis), it would be great.

That release should be implemented on the current master branch because it will benefit the Rails 4 version too

tagliala added a commit that referenced this issue Dec 27, 2016
Removes:
* Ruby < 2.0 code
* Rails 4 code and gemfiles
* MiniTest 4 support

Regenerate javascript
Refers to #645
tagliala added a commit that referenced this issue Dec 27, 2016
tagliala added a commit that referenced this issue Dec 27, 2016
Removes:
* Ruby < 2.0 code
* Rails 4 code and gemfiles
* MiniTest 4 support

Regenerate javascript
Refers to #645
tagliala added a commit that referenced this issue Dec 27, 2016
tagliala added a commit that referenced this issue Jan 1, 2017
Removes:
* Ruby < 2.0 code
* Rails 4 code and gemfiles
* MiniTest 4 support

Regenerate javascript
Refers to #645
tagliala added a commit that referenced this issue Jan 1, 2017
tagliala added a commit that referenced this issue Jan 19, 2017
Removes:
* Ruby < 2.0 code
* Rails 4 code and gemfiles
* MiniTest 4 support

Regenerate javascript
Refers to #645
tagliala added a commit that referenced this issue Jan 19, 2017
tagliala added a commit to DavyJonesLocker/client_side_validations-simple_form that referenced this issue Jan 19, 2017
tagliala added a commit that referenced this issue Jan 19, 2017
tagliala added a commit that referenced this issue Jan 19, 2017
* rails5:
  Fix package.json
  Avoid duplication in form builder
  Support case insensitive confirmation validator Refers to #645
  Use Method#prepend instead of alias_method_chain
  Fix uniqueness validator
  Fix and improve tests
  Remove old code and fix rubocop offenses
  First commit
@tagliala
Copy link
Contributor Author

Master branch is now compatible with Rails 5.

Expect a new release very soon.

Version will be 6.0.0, I will jump 5.x because now CSV will follow its own versioning for proper SemVer

@tagliala
Copy link
Contributor Author

CSV 6.0.0 released. I'm going for CSV-simple_form

@tagliala tagliala added this to the 6.0.0 milestone Jan 20, 2017
@dreyks
Copy link

dreyks commented Jan 20, 2017

Great news! Highly appreciated, thanks!

@rubydev
Copy link

rubydev commented Jan 20, 2017

@tagliala Hi, could you please update CSV-mongoid for this version (and mongoid 6 and rails 5)?

@tagliala
Copy link
Contributor Author

@rubydev I will give it a try, it is the next on the list!

btw you could give a try to https://github.com/DavyJonesLocker/client_side_validations-mongoid/tree/6-0-dev

@tagliala
Copy link
Contributor Author

@rubydev CSV-Mongoid 7.0.0 released

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

No branches or pull requests

8 participants