Skip to content

Commit 114fe12

Browse files
authored
Update collections.md
1 parent 4d8039d commit 114fe12

File tree

1 file changed

+1
-6
lines changed

1 file changed

+1
-6
lines changed

graph/articles/collections.md

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -340,7 +340,7 @@ Take the following model with an entity type `application` that has a collection
340340
</ComplexType>
341341
```
342342
and a scenario arises that requires, for example, to remove individual `keyCredential`s from the collection.
343-
There are two options forward: //// TODO do we want to offer both options, or just one?
343+
There are two options forward:
344344

345345
### 11.1 Side-by-side collection properties (for any collection of structural types)
346346

@@ -419,9 +419,6 @@ HTTP/1.1 400 Bad Request
419419
}
420420
```
421421

422-
TODO should this be a 409 conflict instead?
423-
TODO implement this in WebApi
424-
425422
### 11.2 Redefine as Entity Type (for collections of complex types)
426423

427424
The model can be updated to simply switch the complex type for an entity type:
@@ -474,5 +471,3 @@ GET /applications/{applicationId}?$select=keyCredentials
474471

475472
2. The default behavior for structural collections is to include them in the response payload for their containing entity. If this was the behavior of `application` before, it must be preserved by **auto-expanding** the `keyCredentials` property now that it is a navigation property (because the default behavior for navigation properties is to **not** expand them).
476473
3. Structural collections can be updated using a `PATCH` request to the containing entity to replace the entire contents of the collection. If the service supported such updates to the structural collection, then updates to the new navigation property must preserve this behavior.
477-
478-
TODO implement this in webapi

0 commit comments

Comments
 (0)