best_in_place icon indicating copy to clipboard operation
best_in_place copied to clipboard

Remove rails 6 deprecations

Open JDrizzy opened this issue 4 years ago • 9 comments

  • Fix rails 6 deprecations
  • Attempt to fix tests
  • Added rails 5.1 & 6 and ruby versions 2.5.7 & 2.6.5 to tests
    • Removed rails 3.2, 4.0 and 4.1

Resolves https://github.com/bernat/best_in_place/issues/623 Resolves https://github.com/bernat/best_in_place/issues/621 & https://github.com/bernat/best_in_place/issues/611

JDrizzy avatar Mar 12 '20 09:03 JDrizzy

@JDrizzy awesome job. To make the tests pass for Rails 4.2 work you probably need to pin rspec-rails to 3.x. and bundler to 1.x.

dirkjonker avatar Apr 08 '20 15:04 dirkjonker

By the way, I can confirm that this works with our Rails 5.2 in production. Also works after upgrading to Rails 6 in my development environment.

dirkjonker avatar Apr 08 '20 15:04 dirkjonker

Does it work with web packer?

evanliewer avatar Apr 09 '20 18:04 evanliewer

Does it work with web packer?

I'm currently using it on a production app with webpacker, but the library's assets are still being provided by sprockets, unfortunately.

JDrizzy avatar Apr 14 '20 04:04 JDrizzy

Hi, I'm not sure how to solve the issue. What should I do to make work with rails 6? someone gave me the link to this page but what steps should I take or follow. Any tips?

Thanks,

Amm1r-IT avatar Apr 16 '20 21:04 Amm1r-IT

@Amm1r-IT to use the changes for this PR you can either reference my repository in your gemfile:

gem 'best_in_place', github: 'JDrizzy/best_in_place'

Or, fork your own copy and reference that instead.

Ideally, this PR should be merged so you don't need to complete the above two workarounds but I'm not sure when that will be.

JDrizzy avatar Apr 16 '20 22:04 JDrizzy

Thanks man

Amm1r-IT avatar Apr 20 '20 02:04 Amm1r-IT

What's the status ?

dsounded avatar May 26 '20 11:05 dsounded

When this PR is planned to be merged?

ArturDatsenko avatar Apr 02 '21 11:04 ArturDatsenko