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

Implemented support for MKCOL method #465

Merged
merged 2 commits into from
Mar 23, 2016
Merged

Conversation

nj
Copy link
Contributor

@nj nj commented Mar 11, 2016

MKCOL - part of the HTTP Extensions for Distributed Authoring (also known as WebDAV) - allow creation of a new collection resource at the location specified.

By extending with the MKCOL method, will cover (most) basic WebDAV implementation and make httparty more generic and increase the fun 😄

MKCOL - part of the HTTP Extensions for Distributed Authoring (also known as WebDAV) - allow creation of a new collection resource at the location specified.

By extending with MKCOL method, will cover basic WebDAV implementation and make httparty more generic and increase the fun.
@@ -9,7 +9,8 @@ class Request #:nodoc:
Net::HTTP::Head,
Net::HTTP::Options,
Net::HTTP::Move,
Net::HTTP::Copy
Net::HTTP::Copy,
Net::HTTP::Mkcol
Copy link
Owner

Choose a reason for hiding this comment

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

Would you add a trailing comma here so we don't end up with a diff like this next time something is added?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Had actually thought about it, but decided to avoid in the first place to keep konsistency with the original.

It has been added now in 5f94172 😃

@jnunemaker
Copy link
Owner

Seems fine. I know nothing about it, so I guess we'll see how it goes with maintenance on it going forward.

@jnunemaker jnunemaker merged commit 8a66ffc into jnunemaker:master Mar 23, 2016
@jnunemaker
Copy link
Owner

Thanks!

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