Add TODO wrt to CommandValidation.invalidArgument

digital-asset · Oct 20, 2021 · 2007e80 · 2007e80
1 parent 8a96248
commit 2007e80
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/...ommon/src/main/scala/com/digitalasset/platform/server/api/validation/ErrorFactories.scala b/...ommon/src/main/scala/com/digitalasset/platform/server/api/validation/ErrorFactories.scala
@@ -123,6 +123,8 @@ class ErrorFactories private (errorCodesVersionSwitcher: ErrorCodesVersionSwitch
         addDefiniteAnswerDetails(definiteAnswer, statusBuilder)
         grpcError(statusBuilder.build())
       },
+      // TODO error codes: This error group is confusing for this generic error as it can be dispatched
+      //                   from call-sites that do not involve command validation (e.g. ApiTransactionService).
       v2 = LedgerApiErrors.CommandValidation.InvalidArgument
         .Reject(message)
         .asGrpcError,