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

Pr/send file mime : Support send_file("file", content_type => "png") #405

Closed
wants to merge 23 commits into from
Closed

Pr/send file mime : Support send_file("file", content_type => "png") #405

wants to merge 23 commits into from

Conversation

ambs
Copy link
Member

@ambs ambs commented Mar 18, 2011

As discussed in IRC with bigpresh. Lets the user force a content_type programatically

xsawyerx and others added 23 commits March 14, 2011 09:40
…stalled.

(note: DSC IS broken, so don't merge this until it's fixed)
Signed-off-by: Michael G. Schwern <schwern@pobox.com>
* review/schwern:
  changes
  Fix the cookbook docs on using the Debug Plack middleware.
It was checked at load time in Dancer::Session::Abstract and then never again.

This allows one to maintain sessions for multiple sites in one Dancer process.
... in which case, the method is assumed to be GET
Signed-off-by: franck cuny <franck@lumberjaph.net>
* review/yannick:
  update CHANGES
  update CHANGES
  add documentation
  first argument of tests can be a string
Signed-off-by: franck cuny <franck@lumberjaph.net>
* review/yannick:
  update CHANGES
  update CHANGES
  add documentation
  first argument of tests can be a string

Conflicts:
	CHANGES

Signed-off-by: franck cuny <franck@lumberjaph.net>
* review/racke:
  add bold to POD; reflect changes
  Extend the POD on plugin_setting to prevent a pitfall with plugin modules more than 3 levels deep.
@sukria
Copy link
Member

sukria commented Mar 18, 2011

approved for the PR, but it appears it has not been done as described in Dancer::Development, because you have based your PR on the master branch, whereas it should be based on our "devel".

This is done via the GitHub web interface when you create the PR. Can you redo it please? :)

Thanks!

@sukria
Copy link
Member

sukria commented Mar 18, 2011

I close this one ;)

@sukria sukria closed this Mar 18, 2011
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.

8 participants