Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add block argument for sort_link #604

Merged
merged 1 commit into from
Dec 23, 2015

Conversation

dalpo
Copy link
Contributor

@dalpo dalpo commented Oct 6, 2015

Allow passing block argument to the sort_link method:

<%= sort_link(@q, :name, [:name, 'kind ASC']) do %>
  <strong>Player Name</strong>
<% end %>

@dalpo dalpo force-pushed the feature/sort-link-block branch from 4377fd5 to 2e6667b Compare October 6, 2015 23:10
@dalpo
Copy link
Contributor Author

dalpo commented Oct 7, 2015

Uhm.. Failures don't depend on my commit 🐛

It seems that the rack-cache gem has been changed and now require at least ruby 2.0

Gem::InstallError: rack-cache requires Ruby version >= 2.0.0.
Using activesupport 3.2.22 from git://github.com/rails/rails.git (at 3-2-stable)
An error occurred while installing rack-cache (1.3.0), and Bundler cannot
continue.

btw, let me know what do you think about this PR 😉
Thanks

@jonatack
Copy link
Contributor

jonatack commented Oct 8, 2015

Thanks! Will have a look soon.

jonatack added a commit that referenced this pull request Dec 23, 2015
@jonatack jonatack merged commit 997b856 into activerecord-hackery:master Dec 23, 2015
@jonatack
Copy link
Contributor

@dalpo Very sorry for not getting back to your PR much sooner. It looks good 💜 🚀

Would you like to make a PR to update the README and Change Log?

@dalpo dalpo mentioned this pull request Dec 27, 2015
@dalpo
Copy link
Contributor Author

dalpo commented Dec 27, 2015

Sure! Here you can find the PR #630

jonatack added a commit that referenced this pull request Jan 1, 2016
[skip ci] Update changelog (follow-up to #604).
dalpo added a commit to dalpo/ransack that referenced this pull request Jan 2, 2016
jonatack added a commit that referenced this pull request Jan 2, 2016
jonatack added a commit that referenced this pull request Jan 13, 2016
Use main repository link in CHANGELOG.md. Follow-up to #604.
avit added a commit to avit/ransack that referenced this pull request Feb 19, 2016
* master: (133 commits)
  Use main repository link in CHANGELOG.md.
  Update documentation with Mongoid 5 support.
  Support Mongoid 5.
  Remove inline conditional from constructor back to private method
  Rails 5 params: instance_variable_get -> to_unsafe_h
  Add README doc and fix CHANGELOG for PR activerecord-hackery#604 [skip ci]
  Travis-ci: expect Rails 3.1/3.2 with Ruby 2.3.0 to pass.
  This was moved from Squeel to Polyamorous [skip ci]
  Update changelog
  Gemfile: bump Mongoid back down to v4
  Wrap lines at 80 chars
  Fix specs broken by Rails 3 on Ruby 2.3.0.
  Do not run the datetime_select spec when Rails 3.1 && Ruby 2.3.0
  Gemfile: bump Mongoid from 4 to 5
  Allow Travis failures for Rails 3 with Ruby 2.3 until debugged
  Test all versions with Ruby 2.3.0
  Remove copyright from README; it is already in the LICENSE file.
  Remove unneeded `before` block test setup
  Simplify doc comment, wraps to 80 chars [skip ci]
  Fix Adapters::ActiveRecord::Context for Rails 5
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants