Skip to content

Allow custom content-type without setting a filename.#138

Merged
alexindigo merged 2 commits intoform-data:masterfrom
DylanPiercey:issue-74
Jul 30, 2015
Merged

Allow custom content-type without setting a filename.#138
alexindigo merged 2 commits intoform-data:masterfrom
DylanPiercey:issue-74

Conversation

@DylanPiercey
Copy link
Copy Markdown
Member

Resolves: #74.

Comment thread lib/form_data.js
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add custom content-type for http streams too? Thank you.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will get to this tomorrow :).

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

@alexindigo
Copy link
Copy Markdown
Member

Thanks a lot for the PR, looks very nice. Just few minor changes and we'd hit home run. :)

@DylanPiercey
Copy link
Copy Markdown
Member Author

Let me know if any other tweaks are needed.

alexindigo added a commit that referenced this pull request Jul 30, 2015
Allow custom content-type without setting a filename.
@alexindigo alexindigo merged commit 6e6a289 into form-data:master Jul 30, 2015
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