Added test for sort by price and route and contoller, - Githubissues

CCirbo / Little_Shop

1 stars 2 forks source link

Added test for sort by price and route and contoller, #50

Closed MDelarosa1993 closed 3 weeks ago

MDelarosa1993 commented 3 weeks ago

This PR introduces sorting functionality to the ItemsController:

Controller Update: Modifies index action in ItemsController to optionally sort items by unit_price when the sorted parameter is set to "price".
Model Update: Adds sorted_by_price class method in the Item model to handle sorting by price.
Routes: Ensures that the items#index action can handle sorting via query parameters.
Tests: Includes tests for verifying sorting functionality by price in both model and request specs.

This update enables fetching items sorted by price from low to high, improving the API's capability to deliver ordered item lists.

jimmacur commented 3 weeks ago

Melchor, great job on this! I noticed you’ve added sorting by price to the items index action—love that feature. The changes to the controller, model, and routes all look solid. The test updates for sorting by price are also a nice touch. Everything seems to be in order. Thanks for your work on this!