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

:priority option broken #6

Closed
wants to merge 1 commit into from
Closed

:priority option broken #6

wants to merge 1 commit into from

Conversation

onedanshow
Copy link

Small bug fix. I found that the "options.stringify_keys!" would break search for the :priority symbol in the options hash.

@jim
Copy link
Collaborator

jim commented Jun 5, 2012

Thanks for finding this. I'm going to add a test to cover this, and change this line to use a string key.

@jim jim closed this in 219b8e9 Jun 5, 2012
jesusdp pushed a commit to designerpages/carmen-rails that referenced this pull request May 27, 2020
Fixes tests:
- Add Ruby 2.1 - 2.6 and Rails 5, 6 to Travis matrix
- Add Rails 5, 6 to Appraisals config
- Fixes implementation for form helper tests
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