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

PUT /person/:id - 405 Method Not Allowed error #20

Closed
Tracked by #32
ikbensiep opened this issue Sep 27, 2022 · 1 comment · Fixed by #34
Closed
Tracked by #32

PUT /person/:id - 405 Method Not Allowed error #20

ikbensiep opened this issue Sep 27, 2022 · 1 comment · Fixed by #34
Assignees
Labels
bug Something isn't working
Milestone

Comments

@ikbensiep
Copy link

ikbensiep commented Sep 27, 2022

When updating a person's display name, the api will log 405 Method Not Allowed.

=> (10) PUT /person/3

{
  type: null,
  httpStatus: 405,
  title: 'Method Not Allowed',
  detail: 'PUT is not allowed',
  instance: null,
  allow: [ 'GET', 'OPTIONS', 'POST' ]
}
@ikbensiep ikbensiep changed the title 405 Method Not Allowed errors PUT /person/:id - 405 Method Not Allowed error Sep 27, 2022
@evert evert added the bug Something isn't working label Sep 28, 2022
@syedfkabir
Copy link
Contributor

syedfkabir commented Sep 28, 2022

Hey @evert, as I was testing this, and looking through the API,
it seems the Person controller doesn't have a PUT method.
It has a POST performing like a PUT. Not sure if this was intentional, I can change it to a PUT.
I have an idea why though.

Meaning a super quick fix @ikbensiep is just changing that code ^ from PUT to POST should work

@BeckyPollard BeckyPollard added this to the MVP 0.5 milestone Sep 30, 2022
syedfkabir added a commit that referenced this issue Oct 17, 2022
PUT /person/:id - 405 Method Not Allowed error #20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants