Skip to content
This repository was archived by the owner on Nov 9, 2017. It is now read-only.

Conversation

@rstacruz
Copy link
Collaborator

As noted in many issues (#61 #62), Turbolinks 5 isn't supported.

@kossnocorp
Copy link
Owner

👍


gem.add_dependency 'railties', '>= 3.1.0'
gem.add_dependency 'turbolinks'
gem.add_dependency 'turbolinks', '< 4'

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is it not < 5?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

V4 doesn't exist haha

On Mon, Jul 18, 2016, 12:34 PM Aleksey Ivanov [email protected]
wrote:

In jquery-turbolinks.gemspec
#63 (comment)
:

@@ -25,5 +25,5 @@ Gem::Specification.new do |gem|
gem.rubygems_version = '1.8.15'

gem.add_dependency 'railties', '>= 3.1.0'

  • gem.add_dependency 'turbolinks'
  • gem.add_dependency 'turbolinks', '< 4'

Why is it not < 5?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kossnocorp/jquery.turbolinks/pull/63/files/c40b554104c85b67df632d079d82993beb91563f#r71099904,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEikVp6qPS7fLY_xKuymjXVxGPccBKAks5qWwJVgaJpZM4JOYX4
.

Copy link

@ivanovaleksey ivanovaleksey Jul 18, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So as V3)
I added < 5 and it installed 2.1.0 (if I remember right).
UPDATE 1
rubygems says last non5 version is 2.5.3.
UPDATE 2
I was wrong.
gem 'turbolinks', '< 5' installed 2.5.3 as it should be.

@glebm
Copy link

glebm commented Oct 31, 2016

Can this be merged and a new version released?

@ivanovaleksey
Copy link

@rstacruz can you merge the PR?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants