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

DELETE method not available #75

Closed
fabianlupa opened this issue Nov 28, 2015 · 1 comment
Closed

DELETE method not available #75

fabianlupa opened this issue Nov 28, 2015 · 1 comment

Comments

@fabianlupa
Copy link
Contributor

Hello,

any specific reason why method="delete" is not supported? I want to use this element to bind to a REST service, which by convention uses the delete-method on deletion-routes.

I saw patch was added at a later point as well ( #53 ) so I am wondering if it is just an oversight or if there is a technical reason.

@rnicholus
Copy link
Owner

Looks like this method support may have been lost in one of the more recent
major version updates. There is no unit test for this method either, which
is surprising. I'll gladly look at a pull request that re-introduces this
support, with at least one simple unit test.
On Sat, Nov 28, 2015 at 10:49 AM Fabian Lupa notifications@github.com
wrote:

Hello,

any specific reason why method="delete" is not supported? I want to use
this element to bind to a REST service, which by convention uses the
delete-method on deletion-routes.

I saw patch was added at a later point as well ( #53
#53 ) so I am wondering if
it is just an oversight or if there is a technical reason.


Reply to this email directly or view it on GitHub
#75.

rnicholus added a commit that referenced this issue Nov 29, 2015
@rnicholus rnicholus added this to the 2.1.1 milestone Nov 29, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants