able to use with rails 4.2.1 too

[mpakus]

No description provided.

[yan-hoose]

Since there is a bug in Rails 4.2 that causes Forem to double-increment views, I don't recommend to use Forem with Rails 4.2 until that bug is fixed (#640).

If you are fine with that, go ahead! 😄

[radar]

I am not fine with having Forem support a version of Rails with a known regression. Therefore I won't be accepting this PR until Rails 4.2.2 is released.

[senych-vitalii]

hey @radar Rails 4.2.2 is already released, can you accept PR?
Currently I cannot use Forem without this fix because I use 4.2.1 version of Rails...

[radar]

Does 4.2.2 contain the specified patch or is it just security releases?

On 19 Jun 2015, at 19:43, Vitaliy Senich [email protected] wrote:

hey @radar Rails 4.2.2 is already released, can you accept PR?

—
Reply to this email directly or view it on GitHub.

[senych-vitalii]

@radar it seems not...
Sorry, I missed that counter is working in 4.2 version and is not working in 4.2.1.

[yan-hoose]

Actually counter caches are broken in all of Rails 4.2 versions. Forem Rails dependency should be

s.add_dependency 'rails', '~> 4.0', '< 4.2'

instead of

s.add_dependency 'rails', '~> 4.0', '<= 4.2'

[radar]

I'm fine with leaving the dependency as it is now. If people complain about the counter cache issue, we can point them to this issue.

[edmondchui]

The counter issue is fixed in Rails 4.2.4. Could you please update the dependency to Rails 4.2.4, 4.2.5, and above? Thanks.

[sintro]

The counter issue is fixed in Rails 4.2.4. Could you please update the dependency to Rails 4.2.4, 4.2.5, and above? Thanks.

Is not it already updated?