Skip to content

Resource identifier should be placed as a path parameter #25

Open
@wypiorsebastian

Description

@wypiorsebastian

I know it's not a course about RESTful principles but maybe you'll consider in the future extracting resource id from request body to path parameter ?

public async Task<IActionResult> UpdateProduct([FromBody] Product product)

Activity

mehmetozkaya

mehmetozkaya commented on May 20, 2021

@mehmetozkaya
Member

yes you exactly right, this is good to fix in this issue. I will check it later thanks.

wypiorsebastian

wypiorsebastian commented on May 20, 2021

@wypiorsebastian
Author

No problem. Glad I could help. BTW - great course :)

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

      Development

      No branches or pull requests

        Participants

        @mehmetozkaya@wypiorsebastian

        Issue actions

          Resource identifier should be placed as a path parameter · Issue #25 · aspnetrun/run-aspnetcore-microservices