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

Improve Method Naming for Clarity in /api/v1/foods/{id} Endpoint #24

Open
iamcymentho opened this issue Oct 24, 2023 · 1 comment
Open

Comments

@iamcymentho
Copy link

Issue Description

Problem:

In the /api/v1/foods/{id} endpoint, we currently have methods with similar names (GET, PUT, DELETE, PATCH) which can be confusing for developers, especially those who are new to developing .NET Web APIs. Clarity in method naming is essential for enhancing the project's accessibility and learning experience.

Proposal:

I propose to improve the method names in the /api/v1/foods/{id} endpoint to make them more understandable and tailored towards the action being performed. This will not only enhance the project's usability but also provide a more educational experience for developers who are new to .NET Web API development.

Action Plan:

I will work on renaming the methods in the /api/v1/foods/{id} endpoint as follows:

Change GET to GetFoodById
Change PUT to UpdateFood
Change DELETE to DeleteFood
Change PATCH to ModifyFood

These changes will align with RESTful API conventions and provide a clear and intuitive understanding of each method's purpose.

Benefits:

  • Improved clarity in method names.
  • Enhanced learning experience for newcomers to .NET Web API development.
  • Better alignment with RESTful API conventions.

Permission Request:

I kindly request permission to work on this issue and make the proposed changes. I believe that this enhancement will benefit both the project and the community of developers using it.

@FabianGosebrink
Copy link
Owner

Hey thanks. Sure, go for it. Happy to review 🙂

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

No branches or pull requests

2 participants