Skip to content

Support registering OPTIONS HTTP method handlers via RouteTableDef#4615

Merged
webknjaz merged 2 commits intoaio-libs:masterfrom
meetmangukiya:route-table-deff
Mar 26, 2020
Merged

Support registering OPTIONS HTTP method handlers via RouteTableDef#4615
webknjaz merged 2 commits intoaio-libs:masterfrom
meetmangukiya:route-table-deff

Conversation

@meetmangukiya
Copy link
Copy Markdown
Contributor

@meetmangukiya meetmangukiya commented Mar 8, 2020

What do these changes do?

Allows to register Options method handlers for resources. Currently it gives an AttributeError: 'RouteTableDef' object has no attribute 'options'

Are there changes in behavior for the user?

Related issue number

Resolves #4663

Checklist

  • I think the code is well written
  • Unit tests for the changes exist
  • Documentation reflects the changes
  • If you provide code modification, please add yourself to CONTRIBUTORS.txt
    • The format is <Name> <Surname>.
    • Please keep alphabetical order, the file is sorted by names.
  • Add a new news fragment into the CHANGES folder
    • name it <issue_id>.<type> for example (588.bugfix)
    • if you don't have an issue_id change it to the pr id after creating the pr
    • ensure type is one of the following:
      • .feature: Signifying a new feature.
      • .bugfix: Signifying a bug fix.
      • .doc: Signifying a documentation improvement.
      • .removal: Signifying a deprecation or removal of public API.
      • .misc: A ticket has been closed, but it is not of interest to users.
    • Make sure to use full sentences with correct case and punctuation, for example: "Fix issue with non-ascii contents in doctest text files."

@meetmangukiya meetmangukiya requested a review from asvetlov as a code owner March 8, 2020 16:10
@asvetlov

This comment has been minimized.

@codecov-io

This comment has been minimized.

@webknjaz
Copy link
Copy Markdown
Member

@meetmangukiya please add tests and a change fragment

@meetmangukiya meetmangukiya requested a review from webknjaz as a code owner March 26, 2020 06:52
@psf-chronographer psf-chronographer bot added the bot:chronographer:provided There is a change note present in this PR label Mar 26, 2020
@meetmangukiya
Copy link
Copy Markdown
Contributor Author

@webknjaz added the tests

@webknjaz webknjaz changed the title web_routedef: Allow RouteTableDef to register options method handlers Support registering OPTIONS HTTP method handlers via RouteTableDef Mar 26, 2020
@webknjaz webknjaz merged commit fa52a26 into aio-libs:master Mar 26, 2020
asvetlov pushed a commit that referenced this pull request Oct 16, 2020
asvetlov pushed a commit that referenced this pull request Nov 19, 2021
asvetlov pushed a commit that referenced this pull request Nov 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bot:chronographer:provided There is a change note present in this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

RouteTableDef can't register OPTIONS request handlers

4 participants