From 8ae9327c96c25efd63e4d4a0d3fd2e11c8cba344 Mon Sep 17 00:00:00 2001
From: MohammadAhmdi <mohammadahmadidv@gmail.com>
Date: Mon, 19 Oct 2026 23:55:57 +0330
Subject: [PATCH] docs(result): document QuickbaseResult API

---
 .../Helpers/QuickbaseCommandBuilder.cs        |  4 +-
 QuickbaseNet/QuickbaseNet.csproj              |  3 +-
 QuickbaseNet/QuickbaseResult.cs               | 61 ++++++++++++++++++-
 3 files changed, 63 insertions(+), 5 deletions(-)

diff --git a/QuickbaseNet/Helpers/QuickbaseCommandBuilder.cs b/QuickbaseNet/Helpers/QuickbaseCommandBuilder.cs
index dbc6330..df672cc 100644
--- a/QuickbaseNet/Helpers/QuickbaseCommandBuilder.cs
+++ b/QuickbaseNet/Helpers/QuickbaseCommandBuilder.cs
@@ -128,8 +128,8 @@ public RecordBuilder AddField<T>(int fieldId, T value)
             /// <summary>
             /// Adds multiple fields to the record being built.
             /// </summary>
-            /// <param name="fields"></param>
-            /// <returns></returns>
+            /// <param name="fields">The field IDs and values to add to the record.</param>
+            /// <returns>The current <see cref="RecordBuilder"/> instance.</returns>
             public RecordBuilder AddFields(params (int fieldId, dynamic value)[] fields)
             {
                 foreach (var field in fields)
diff --git a/QuickbaseNet/QuickbaseNet.csproj b/QuickbaseNet/QuickbaseNet.csproj
index 2c5fa64..25e1c4c 100644
--- a/QuickbaseNet/QuickbaseNet.csproj
+++ b/QuickbaseNet/QuickbaseNet.csproj
@@ -25,8 +25,7 @@
 		<RepositoryType>git</RepositoryType>
 		<!-- <PackageReleaseNotes>Initial release; right now only have Query capabilities available.</PackageReleaseNotes> -->
 		<GenerateDocumentationFile>true</GenerateDocumentationFile>
-		<NoWarn>$(NoWarn);1591</NoWarn>
 		<PackageLicenseExpression>MIT</PackageLicenseExpression>
 	</PropertyGroup>
 
-</Project>
\ No newline at end of file
+</Project>
diff --git a/QuickbaseNet/QuickbaseResult.cs b/QuickbaseNet/QuickbaseResult.cs
index 45cf79c..213d3da 100644
--- a/QuickbaseNet/QuickbaseResult.cs
+++ b/QuickbaseNet/QuickbaseResult.cs
@@ -3,8 +3,17 @@
 
 namespace QuickbaseNet
 {
+    /// <summary>
+    /// Represents the outcome of a Quickbase operation.
+    /// </summary>
     public class QuickbaseResult
     {
+        /// <summary>
+        /// Initializes a new instance of the <see cref="QuickbaseResult"/> class.
+        /// </summary>
+        /// <param name="isSuccess">Whether the operation succeeded.</param>
+        /// <param name="quickbaseError">The error associated with a failed operation, or <see cref="QuickbaseError.None"/> on success.</param>
+        /// <exception cref="ArgumentException">The success state and error value are inconsistent.</exception>
         protected internal QuickbaseResult(bool isSuccess, QuickbaseError quickbaseError)
         {
             if (isSuccess && quickbaseError != QuickbaseError.None ||
@@ -17,37 +26,87 @@ protected internal QuickbaseResult(bool isSuccess, QuickbaseError quickbaseError
             QuickbaseError = quickbaseError;
         }
 
+        /// <summary>
+        /// Gets a value indicating whether the operation succeeded.
+        /// </summary>
         public bool IsSuccess { get; }
 
+        /// <summary>
+        /// Gets a value indicating whether the operation failed.
+        /// </summary>
         public bool IsFailure => !IsSuccess;
 
+        /// <summary>
+        /// Gets the error associated with the operation, or <see cref="QuickbaseError.None"/> when it succeeded.
+        /// </summary>
         public QuickbaseError QuickbaseError { get; }
 
+        /// <summary>
+        /// Creates a successful result without a value.
+        /// </summary>
+        /// <returns>A successful result.</returns>
         public static QuickbaseResult Success() => new QuickbaseResult(true, QuickbaseError.None);
 
+        /// <summary>
+        /// Creates a successful result containing the specified value.
+        /// </summary>
+        /// <typeparam name="TValue">The type of the result value.</typeparam>
+        /// <param name="value">The value to include in the result.</param>
+        /// <returns>A successful result containing <paramref name="value"/>.</returns>
         public static QuickbaseResult<TValue> Success<TValue>(TValue value) => new QuickbaseResult<TValue>(value, true, QuickbaseError.None);
 
+        /// <summary>
+        /// Creates a failed result without a value.
+        /// </summary>
+        /// <param name="quickbaseError">The error that caused the operation to fail.</param>
+        /// <returns>A failed result containing <paramref name="quickbaseError"/>.</returns>
         public static QuickbaseResult Failure(QuickbaseError quickbaseError) => new QuickbaseResult(false, quickbaseError);
 
+        /// <summary>
+        /// Creates a failed result with the specified value type.
+        /// </summary>
+        /// <typeparam name="TValue">The type of value the result would contain on success.</typeparam>
+        /// <param name="quickbaseError">The error that caused the operation to fail.</param>
+        /// <returns>A failed result containing <paramref name="quickbaseError"/>.</returns>
         public static QuickbaseResult<TValue> Failure<TValue>(QuickbaseError quickbaseError) => new QuickbaseResult<TValue>(default, false, quickbaseError);
     }
 
+    /// <summary>
+    /// Represents the outcome of a Quickbase operation that returns a value.
+    /// </summary>
+    /// <typeparam name="TValue">The type of value returned by a successful operation.</typeparam>
     public class QuickbaseResult<TValue> : QuickbaseResult
     {
         private readonly TValue _value;
 
+        /// <summary>
+        /// Initializes a new instance of the <see cref="QuickbaseResult{TValue}"/> class.
+        /// </summary>
+        /// <param name="value">The value returned by a successful operation.</param>
+        /// <param name="isSuccess">Whether the operation succeeded.</param>
+        /// <param name="quickbaseError">The error associated with a failed operation, or <see cref="QuickbaseError.None"/> on success.</param>
+        /// <exception cref="ArgumentException">The success state and error value are inconsistent.</exception>
         protected internal QuickbaseResult(TValue value, bool isSuccess, QuickbaseError quickbaseError)
             : base(isSuccess, quickbaseError)
         {
             _value = value;
         }
 
+        /// <summary>
+        /// Gets the value returned by a successful operation.
+        /// </summary>
+        /// <exception cref="InvalidOperationException">The result represents a failed operation.</exception>
         public TValue Value => IsSuccess
             ? _value
             : throw new InvalidOperationException("The value of a failure result can't be accessed.");
 
+        /// <summary>
+        /// Converts a value to a successful result, or a null value to a failed result.
+        /// </summary>
+        /// <param name="value">The value to convert.</param>
+        /// <returns>A successful result containing <paramref name="value"/>, or a failed result with <see cref="QuickbaseError.NullValue"/> when <paramref name="value"/> is null.</returns>
         public static implicit operator QuickbaseResult<TValue>(TValue value) =>
             !(value == null) ? Success(value) : Failure<TValue>(QuickbaseError.NullValue);
     }
 
-}
\ No newline at end of file
+}