You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
nickzelei
changed the title
[NEOS-1688] account_id is not needed on ValidateJobMappings
[NEOS-1688] account_id is not needed on ValidateJobMappingsRequest
Dec 18, 2024
it can be inferred from the connection_id. Should be deprecated. and removed in v0.6.
In the mean time, we should mark it as deprecated in the protos and stop passing it around.
The method already calls
connectionservice.GetConnection
which ensures the calling user is in the connection's account.We can deprecate the field by adding a comment to the top
@deprecated - <explain>
as well as adding a[deprecated = true]
to the value itselfFrom SyncLinear.com | NEOS-1688
The text was updated successfully, but these errors were encountered: