From e5555684676d1d1f9e15b0e1cc7d58df2cb6438a Mon Sep 17 00:00:00 2001 From: Richard Penman <26063146+richardpenman@users.noreply.github.com> Date: Tue, 6 Jun 2023 16:45:25 -0400 Subject: [PATCH 1/2] Update 0162.md The solution for a question in the API readability exam says tagging requires (not accepts) a revision ID, so should this be updated? --- aip/general/0162.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/aip/general/0162.md b/aip/general/0162.md index c8de3e261d..c2ccaa3c5a 100644 --- a/aip/general/0162.md +++ b/aip/general/0162.md @@ -144,8 +144,8 @@ message TagBookRevisionRequest { } ``` -- The `name` field **should** require an explicit revision ID to be provided. - - The field **should** be [annotated as required][aip-203]. +- The `name` field **must** require an explicit revision ID to be provided. + - The field **must** be [annotated as required][aip-203]. - The field **should** identify the [resource type][aip-123] that it references. - The `tag` field **should** be [annotated as required][aip-203]. From b5f07efea22362482d9b54043e91e2de72129f02 Mon Sep 17 00:00:00 2001 From: Richard Penman <26063146+richardpenman@users.noreply.github.com> Date: Fri, 9 Jun 2023 15:37:18 -0400 Subject: [PATCH 2/2] Update 0162.md --- aip/general/0162.md | 1 + 1 file changed, 1 insertion(+) diff --git a/aip/general/0162.md b/aip/general/0162.md index c2ccaa3c5a..77e819eace 100644 --- a/aip/general/0162.md +++ b/aip/general/0162.md @@ -403,6 +403,7 @@ used, and **must** clearly document it. ## Changelog +- **2023-06-09**: Clarified that tagging requires a revision ID. - **2021-04-27**: Added guidance on returning the resource from Delete Revision. [aip-123]: ./0123.md