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

Track should allow hash arguments to be strings #13

Closed
shipstar opened this issue Aug 1, 2013 · 2 comments
Closed

Track should allow hash arguments to be strings #13

shipstar opened this issue Aug 1, 2013 · 2 comments

Comments

@shipstar
Copy link

shipstar commented Aug 1, 2013

I'm enqueuing a tracking job in Redis that consists of a simple hash. When the hash is deserialized, it's no longer a Hash#with_indifferent_access, so the keys come in as strings.

I'm manually calling Hash#with_indifferent_access, but it seems fairly rigid to require the keys to be symbols.

I can submit a pull request if it makes sense.

@calvinfo
Copy link
Contributor

calvinfo commented Aug 1, 2013

I'm not opposed to this idea, I'd like to see how the code for it looks. Definitely submit a pull request. As long as it is clean, backwards compatible, and doesn't add new dependencies, I'm cool with it. :)

@calvinfo
Copy link
Contributor

calvinfo commented Aug 3, 2013

Fixed by #16

@calvinfo calvinfo closed this as completed Aug 3, 2013
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

No branches or pull requests

2 participants