Skip to content
This repository has been archived by the owner on Dec 7, 2018. It is now read-only.

Added a query Hash #100

Closed
wants to merge 1 commit into from
Closed

Added a query Hash #100

wants to merge 1 commit into from

Conversation

Asmod4n
Copy link
Contributor

@Asmod4n Asmod4n commented Sep 25, 2013

Reel was missing a Hash of query values, this looks okayish?

You can access it like request.query['some_value']

Reel was missing a Hash of query values, this looks okayish?

You can access it like request.query['some_value']
@coveralls
Copy link

Coverage Status

Coverage decreased (-3.07%) when pulling 8aa529d on Asmod4n:patch-1 into 5ca565d on celluloid:master.

@halorgium
Copy link
Contributor

This kind of thing is always incorrectly coded. I'd prefer to leave this to another library.
Rack has an implementation which you can require and use.

@halorgium halorgium closed this Sep 25, 2013
@Asmod4n Asmod4n deleted the patch-1 branch September 27, 2013 17:09
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.

None yet

3 participants