forked from AdaGold/solar-system-api
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
updates wave 5 to focus on query params (agrees with learn activity)
- Loading branch information
1 parent
01b6a39
commit 2d0c395
Showing
1 changed file
with
11 additions
and
6 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,9 +1,14 @@ | ||
# Wave 05: Review and Refactor | ||
|
||
Review the requirements for Wave 01 - 04 | ||
* Test the endpoints using postman | ||
* Complete or fix any incomplete or broken endpoints | ||
* Look for opportunities to refactor | ||
## Update Endpoints | ||
|
||
As time allows, add custom routes. | ||
* Consider using query params | ||
Update the read all planets endpoint with similar functionality presented in the Hello Books API: | ||
|
||
As a client, I want to send a request... | ||
|
||
1. ... to get a list of `planet`s, restricted to those with a match in the `description`, so that I can find a planet by a partial description. | ||
2. ... to get a list of `planet`s, restricted to those with some kind of match to your additional attribute (e.g. larger/smaller than some size, partial match of a fun fact, etc), so that I can find planets with similar properties. | ||
|
||
I should be able to combine the effects of the two filters in order to filter results by multiple properties at once. | ||
|
||
If time allows, consider adding a non-filter query parameter, such as sorting by a specific attribute. Remember that sorting can be done by the database, so try not to sort the results in your application code. |