Change grant_management_action "update" to "add" or "append"
Issue #436
resolved
In order to clarify what is meant in section 3.4 with regards to updating the details of a grant using grant_management_action
we propose to change the keyword from “update” to “add” or possibly “append”.
It doesn’t seem possible to do an update that removes privileges (which is possible with “replace”) and supporting proper update would require semantics to be added for each data point to describe how it should be updated (i.e. delete, replace ..) inside the update request.
Therefore we suggest that we tighten the keyword to avoid confusion or incorrect assumptions.
Comments (9)
-
-
OSW catch up update: change to merge
-
merge away
-
Pull request raised: https://bitbucket.org/openid/fapi/pull-requests/336
-
-
assigned issue to
-
assigned issue to
-
- changed milestone to 2nd Implementers Draft
-
New PR: Rename update action to merge
-
- changed status to open
-
- changed status to resolved
PR merged
- Log in to comment
I’ve already raised it some time ago. My proposal that time was to use “merge”. If we were about to make a change, I’d still lean towards “merge”. Please note that currently “merge” appears in the description of the “update” action.