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

Fix test to reflect use of to_param for keys #3065

Merged
merged 1 commit into from
Sep 21, 2011
Merged

Fix test to reflect use of to_param for keys #3065

merged 1 commit into from
Sep 21, 2011

Conversation

brainopia
Copy link
Contributor

No description provided.

@arunagw
Copy link
Member

arunagw commented Sep 20, 2011

/cc @spastorino

spastorino added a commit that referenced this pull request Sep 21, 2011
Fix test to reflect use of to_param for keys
@spastorino spastorino merged commit 95646f4 into rails:master Sep 21, 2011
spastorino added a commit that referenced this pull request Sep 21, 2011
Fix test to reflect use of to_param for keys
shwoodard added a commit to shwoodard/rails-1 that referenced this pull request Sep 22, 2011
* '3-1-stable' of github.com:rails/rails:
  Merge pull request rails#3096 from phuibonhoa/master
  Fix named routes modifying arguments
  Merge pull request rails#3027 from alex3/assets-precompile-regex-fix
  Merge pull request rails#3081 from guilleiguaran/move-precompile-to-class
  Add jquery-rails to Gemfile of plugins, test/dummy app uses it. Closes rails#3091
  copy edits 302e570
  copy edits 908f261
  remove unnecessary markup
  Locale code for Czech is ":cs"
  Document ActiveRecord::QueryMethods#select
  sass-rails helpers - hyphenated in Sass, underscored in Ruby.
  Fixing incorrect notextile tags
  minor edit
  "Passing variables into the translation" section added
  Add reference about --skip-sprockets to Asset Pipeline Guide
  update guides, use html safe translations in i18n
  Merge pull request rails#3065 from brainopia/fix_to_param_test
  fixed test for JRuby. for Sqlite3 in AR-JDBC.It's Jdbc::SQLite3::VERSION
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.

None yet

3 participants