rename to "vp_tokens" from "vp_token" to make clear it is an array
Issue #1336
resolved
Suggest renaming “vp_token“ to "vp_tokens" in the response to make clear it is an array.
Comments (4)
-
reporter -
reporter - changed status to open
On 2021-11-25 SIOP call, we discussed that making it plural
vp_tokens
(orvps_token
) would be good to clarify that it can be an array. -
reporter - changed status to resolved
keeping it
vp_token
since the text clearly says it is an array -
Like
aud
, which can be an array, it’s fine for the name to remain singular. - Log in to comment
initial use-cases seem to be vp_token including only one VP.
+ even when there are multiple VPs, there is still only one VP_token
I am ok with resolving this at the next SIOP call