lnrpc: rename non_succeeded to include_incomplete

lnrpc/rpc.proto

@@ -2171,8 +2171,12 @@ message Payment {
 }
 
 message ListPaymentsRequest {
-    /// Set to also return payments that are not (yet) succeeded.
+    /**
-    bool non_succeeded = 1;
+    If true, then return payments that have not yet fully completed. This means
+    that pending payments, as well as failed payments will show up if this
+    field is set to True.
+    */
+    bool include_incomplete = 1;
 }
 
 message ListPaymentsResponse {

lnrpc/rpc.swagger.json

@@ -940,8 +940,8 @@
         },
         "parameters": [
           {
-            "name": "non_succeeded",
+            "name": "include_incomplete",
-            "description": "/ Set to also return payments that are not (yet) succeeded.",
+            "description": "*\nIf true, then return payments that have not yet fully completed. This means\nthat pending payments, as well as failed payments will show up if this\nfield is set to True.",
             "in": "query",
             "required": false,
             "type": "boolean",

rpcserver.go

@@ -4145,7 +4145,7 @@ func (r *rpcServer) ListPayments(ctx context.Context,
 		// To keep compatibility with the old API, we only return
 		// non-suceeded payments if requested.
 		if payment.Status != channeldb.StatusSucceeded &&
-			!req.NonSucceeded {
+			!req.IncludeIncomplete {
 			continue
 		}