Skip to content

Commit b7bb0c3

Browse files
authored
fix(AIP-164): add missing requirement for delete_time (#1425)
1 parent fa38b8b commit b7bb0c3

1 file changed

Lines changed: 5 additions & 3 deletions

File tree

aip/general/0164.md

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,9 +24,10 @@ mark the resource as having been deleted, but not completely remove it from the
2424
system. If the method behaves this way, it **should** return the updated
2525
resource instead of `google.protobuf.Empty`.
2626

27-
Resources that support soft delete **should** have a `purge_time` field as
28-
described in AIP-148. Additionally, resources **should** include a `DELETED`
29-
state value if the resource includes a `state` field (AIP-216).
27+
Resources that support soft delete **should** have both a `delete_time` and
28+
`purge_time` field as described in AIP-148. Additionally, resources **should**
29+
include a `DELETED` state value if the resource includes a `state` field
30+
(AIP-216).
3031

3132
### Undelete
3233

@@ -147,6 +148,7 @@ resource is not deleted, the service **must** respond with `ALREADY_EXISTS`
147148

148149
## Changelog
149150

151+
- **2024-09-24**: Included missing requirement for `delete_time`.
150152
- **2023-07-13**: Renamed overloaded `expire_time` to `purge_time`.
151153
- **2021-07-12**: Added error behavior when soft deleting a deleted resource.
152154
- **2021-07-12**: Clarified that `ALREADY_EXISTS` errors apply to `Undelete`.

0 commit comments

Comments
 (0)