Skip to content

Commit 67ce103

Browse files
committed
expands description recalling helper methods
- strong hints towards to_dict and validate_model
1 parent 410b762 commit 67ce103

File tree

1 file changed

+10
-0
lines changed

1 file changed

+10
-0
lines changed

ada-project-docs/wave_03.md

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -113,6 +113,10 @@ After I have made the `PATCH` request, I can submit a `GET` request to `/tasks/1
113113
}
114114
```
115115

116+
Notice the same dictionary structure for the Task data as in our wave 1 routes. We should be able to use any response model helper that we are using in other Task routes to help build this response.
117+
118+
Also notice that fundamentally, this route requires that we look up a model by its ID, and then update that model. We should be able to reuse the same route helper methods that we have been using in other Task routes to help build this route.
119+
116120
### Mark Incomplete on an Incomplete Task
117121

118122
Given a task that has:
@@ -143,6 +147,10 @@ After I have made the `PATCH` request, I can submit a `GET` request to `/tasks/1
143147
}
144148
```
145149

150+
Notice the same dictionary structure for the Task data as in our wave 1 routes. We should be able to use any response model helper that we are using in other Task routes to help build this response.
151+
152+
Also notice that fundamentally, this route requires that we look up a model by its ID, and then update that model. We should be able to reuse the same route helper methods that we have been using in other Task routes to help build this route.
153+
146154
## Mark Complete and Mark Incomplete for Missing Tasks
147155

148156
Given that there are no tasks with the ID `1`,
@@ -154,3 +162,5 @@ Then I get a `404 Not Found`.
154162
You may choose the response body.
155163

156164
Make sure to complete the tests for non-existing tasks to check that the correct response body is returned.
165+
166+
Notice that the behavior for a missing Task is consistent with invalid Task IDs in other Task routes. We should be able to use the same route helper methods that we have been using in other Task routes to help build this route.

0 commit comments

Comments
 (0)