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

feat: enable overrideMethod middleware by default #1069

Merged
merged 2 commits into from
Jun 19, 2017

Conversation

fengmk2
Copy link
Member

@fengmk2 fengmk2 commented Jun 19, 2017

only allow override method on POST request by default

#324

Checklist
  • npm test passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)
Description of change

only allow override method on POST request by default

#324
@mention-bot
Copy link

@fengmk2, thanks for your PR! By analyzing the history of the files in this pull request, we identified @popomore, @dead-horse and @shaoshuai0102 to be potential reviewers.

@fengmk2
Copy link
Member Author

fengmk2 commented Jun 19, 2017

依赖 eggjs/koa-override#3 合并发布

@fengmk2
Copy link
Member Author

fengmk2 commented Jun 19, 2017

appveyor 只是 report 问题,test 是过的

@dead-horse
Copy link
Member

windows 下挂了?

@dead-horse
Copy link
Member

合并了

@dead-horse dead-horse merged commit a4ba2a2 into master Jun 19, 2017
@dead-horse dead-horse deleted the enable-override-method branch June 19, 2017 05:48
@fengmk2
Copy link
Member Author

fengmk2 commented Jun 19, 2017

我提个release pr

@fengmk2
Copy link
Member Author

fengmk2 commented Jun 19, 2017

#1027 一起合并再发版本。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants