Merge branch 'dev' into f-insert

include/caosdb/transaction_status.h

@@ -43,6 +43,8 @@ using caosdb::exceptions::TransactionError;
  */
 class TransactionStatus {
 public:
+  // REFACTORING NEEDED: When you touch this code again consider writing a
+  // macro, because this is a lot of redundant code here...
   inline static auto INITIAL() -> const TransactionStatus & {
     static const TransactionStatus initial(
       StatusCode::INITIAL, caosdb::get_status_description(StatusCode::INITIAL));
@@ -61,6 +63,9 @@ public:
   }
   inline static auto RPC_ERROR(const std::string &details)
     -> const TransactionStatus {
+    // We use the GENERIC_RPC_ERROR here because we might want to add further
+    // RPC_ERROR states with different error codes (which stem from GRPC) here
+    // in the future.
     return TransactionStatus(
       StatusCode::GENERIC_RPC_ERROR,
       caosdb::get_status_description(StatusCode::GENERIC_RPC_ERROR) +