NoMethodError on partials when upgrading to rabl 0.11.2 and above about grape-rabl HOT 13 CLOSED

Looks like a legit problem. Maybe you can PR an update to rabl in this project that reproduces the problem, to start?

from grape-rabl.

Sure. I'll prepare a spec for this case if the current specs do not catch
the issue. Probably will be ready on Friday.

On Mon, Nov 24, 2014 at 3:28 PM, Daniel Doubrovkine (dB.) @dblockdotorg <
[email protected]> wrote:

Looks like a legit problem. Maybe you can PR an update to rabl in this
project that reproduces the problem, to start?

—
Reply to this email directly or view it on GitHub
#31 (comment).

רונן מולדובן

מנחה סדנאות, מטפל בתטא-הילינג, מאמן אישי ועסקי מייסד ומנהל בית גשרים
נייד 054-4818159
טלפון 04-6620199
פקס 153-4-6620199
אימייל [email protected]
www.beit-gsharim.com
http://www.beit-gsharim.com
צ'ט Skype: ronen.moldovan MSN: [email protected]
צרו עימי קשר [image: Linkedin]
http://www.linkedin.com/profile?viewProfile=&key=399337&locale=en_US&trk=tab_pro[image:
Facebook] http://www.facebook.com/#%21/ronen.moldovan[image: Twitter]
https://twitter.com/gsharim

--- @ WiseStamp Signature
http://my.wisestamp.com/link?u=4mmzgsymmkn8mh7d&site=www.wisestamp.com/email-install.
Get it now
http://my.wisestamp.com/link?u=4mmzgsymmkn8mh7d&site=www.wisestamp.com/email-install

from grape-rabl.

Same error. Reverting to 0.11.1 resolves it.

from grape-rabl.

@phallstrom @ronenm do you use grape-rabl and sinatra in the same time?

Related: nesquena/rabl#597

from grape-rabl.

@LTe Yes to grape-rabl, No to sinatra.

from grape-rabl.

No not using it with Sinatra.
בתאריך 5 בדצמ 2014 18:14, ‏"Philip Hallstrom" [email protected]
כתב:

@LTe https://github.com/LTe Yes to grape-rabl, No to sinatra.

—
Reply to this email directly or view it on GitHub
#31 (comment).

from grape-rabl.

@ronenm @phallstrom @dblock I created pull request to rabl. This should fix problem.

Related: nesquena/rabl#602

from grape-rabl.

@LTe, I fetched your pull request to a local path and can confirm that the issue has been solved with it.

Waiting for the release of rabl that will include this fix.

Many thanks for the help.

from grape-rabl.

The PR has been merged into rabl! A new release will be issued soon.

from grape-rabl.

Are we saying this can be closed once that is released?

from grape-rabl.

@dblock already released nesquena/rabl@34cd122

Do you think that we should bump dependency for rabl to >= 0.11.5?

from grape-rabl.

Sure. I will close once I check it with the official release.

On Sat, Dec 6, 2014 at 2:02 AM, Daniel Doubrovkine (dB.) @dblockdotorg <
[email protected]> wrote:

Are we saying this can be closed once that is released?

—
Reply to this email directly or view it on GitHub
#31 (comment).

רונן מולדובן

מנחה סדנאות, מטפל בתטא-הילינג, מאמן אישי ועסקי מייסד ומנהל בית גשרים
נייד 054-4818159
טלפון 04-6620199
פקס 153-4-6620199
אימייל [email protected]
www.beit-gsharim.com
http://www.beit-gsharim.com
צ'ט Skype: ronen.moldovan MSN: [email protected]
צרו עימי קשר [image: Linkedin]
http://www.linkedin.com/profile?viewProfile=&key=399337&locale=en_US&trk=tab_pro[image:
Facebook] http://www.facebook.com/#%21/ronen.moldovan[image: Twitter]
https://twitter.com/gsharim

--- @ WiseStamp Signature
http://my.wisestamp.com/link?u=4mmzgsymmkn8mh7d&site=www.wisestamp.com/email-install.
Get it now
http://my.wisestamp.com/link?u=4mmzgsymmkn8mh7d&site=www.wisestamp.com/email-install

from grape-rabl.

Everything's working fine with Rabl 0.11.5

from grape-rabl.