Rails 5 - ClientSide Strategy - Error delegate :env

[GesJeremie]

Hey :)

Just to let you know I'm really glad you did this gem: I encountered all the problems you mentioned in your README and started to make my own implementation before to find mengpaneel (you solved the problem 100 times better than my current scrappy solution).

Even though it's been a while you didn't update the gem it seems to work just fine. I had a minor bug I wanted to report with the class Mengpaneel::Strategy::ClientSide.

Currently you delegate :env to the controller:

delegate :request, :response, :env, to: :controller, allow_nil: true

I'm currently running on Rails 5.2.0 and it seems :env has been deprecated in favor of request.env.

I added a little shortcut in my ApplicationController to solve the problem:

class ApplicationController < ActionController::Base
  include Mengpaneel::Controller

  def env
    request.env
  end
end

If you find the time to do an update, it would be crazy cool!

Thanks again!

[DouweM]

@GesJeremie I’m pleasantly surprised to hear that this 5-year old gem is still proving useful to people! 😄

Thanks for reporting this bug and describing a potential fix; do you think you could submit a pull request? I’m more than happy to release a new version, but I don’t have time to write and verify a fix myself right now. Thanks in advance!