Resolve nullable warnings

src/TableauSharp/Common/Models/ErrorResponse.cs

@@ -5,11 +5,11 @@ namespace TableauSharp.Common.Models;
 public class ErrorResponse
 {
     [JsonPropertyName("code")]
-    public string Code { get; set; }
+    public string Code { get; set; } = string.Empty;
 
     [JsonPropertyName("summary")]
-    public string Summary { get; set; }
+    public string Summary { get; set; } = string.Empty;
 
     [JsonPropertyName("detail")]
-    public string Detail { get; set; }
-}
+    public string Detail { get; set; } = string.Empty;
+}

src/TableauSharp/Embedding/Models/TrustedTicketRequest.cs

@@ -2,7 +2,7 @@ namespace TableauSharp.Embedding.Models;
 
 public class TrustedTicketRequest
 {
-    public string Username { get; set; }
-    public string ClientIp { get; set; }
-    public string TargetSite { get; set; }
-}
+    public string Username { get; set; } = string.Empty;
+    public string? ClientIp { get; set; }
+    public string? TargetSite { get; set; }
+}

src/TableauSharp/Embedding/Models/TrustedTicketResponse.cs

@@ -2,6 +2,6 @@ namespace TableauSharp.Embedding.Models;
 
 public class TrustedTicketResponse
 {
-    public string TicketId { get; set; }
-    public string EmbedUrl { get; set; }
-}
+    public string TicketId { get; set; } = string.Empty;
+    public string EmbedUrl { get; set; } = string.Empty;
+}

src/TableauSharp/Embedding/Services/EmbeddingService.cs

@@ -40,10 +40,14 @@ public async Task<TrustedTicketResponse> GetTrustedTicketAsync(TrustedTicketRequ
 
         var form = new Dictionary<string, string>
         {
-            { "username", request.Username },
-            { "target_site", request.TargetSite }
+            { "username", request.Username }
         };
 
+        if (!string.IsNullOrEmpty(request.TargetSite))
+        {
+            form.Add("target_site", request.TargetSite);
+        }
+
         if (!string.IsNullOrEmpty(request.ClientIp))
         {
             form.Add("client_ip", request.ClientIp);

src/TableauSharp/Permissions/Models/PermissionCapabilityAssignment.cs

@@ -9,5 +9,5 @@ public class PermissionCapabilityAssignment
     public PermissionCapability Capability { get; set; }
 
     [JsonPropertyName("mode")]
-    public string Mode { get; set; } // "Allow" or "Deny"
-}
+    public string Mode { get; set; } = string.Empty; // "Allow" or "Deny"
+}

src/TableauSharp/Permissions/Models/PermissionSyncRequest.cs

@@ -2,6 +2,6 @@ namespace TableauSharp.Permissions.Models;
 
 public class PermissionSyncRequest
 {
-    public string TargetId { get; set; }
+    public string TargetId { get; set; } = string.Empty;
     public List<TableauPermission> Permissions { get; set; } = new();
-}
+}

src/TableauSharp/Permissions/Models/TableauPermission.cs

@@ -5,11 +5,11 @@ namespace TableauSharp.Permissions.Models;
 public class TableauPermission
 {
     [JsonPropertyName("granteeId")]
-    public string GranteeId { get; set; }
+    public string GranteeId { get; set; } = string.Empty;
 
     [JsonPropertyName("granteeType")]
-    public string GranteeType { get; set; } // "User" or "Group"
+    public string GranteeType { get; set; } = string.Empty; // "User" or "Group"
 
     [JsonPropertyName("capabilities")]
     public List<PermissionCapabilityAssignment> Capabilities { get; set; } = new();
-}
+}

src/TableauSharp/Projects/Models/ProjectCreateRequest.cs

@@ -2,7 +2,7 @@ namespace TableauSharp.Projects.Models;
 
 public class ProjectCreateRequest
 {
-    public string Name { get; set; }
-    public string Description { get; set; }
-    public string ParentProjectId { get; set; }
-}
+    public string Name { get; set; } = string.Empty;
+    public string? Description { get; set; }
+    public string? ParentProjectId { get; set; }
+}

src/TableauSharp/Projects/Models/ProjectUpdateRequest.cs

@@ -2,6 +2,6 @@ namespace TableauSharp.Projects.Models;
 
 public class ProjectUpdateRequest
 {
-    public string Name { get; set; }
-    public string Description { get; set; }
-}
+    public string? Name { get; set; }
+    public string? Description { get; set; }
+}

src/TableauSharp/Projects/Models/TableauProject.cs

@@ -5,17 +5,17 @@ namespace TableauSharp.Projects.Models;
 public class TableauProject
 {
     [JsonPropertyName("id")]
-    public string Id { get; set; }
+    public string Id { get; set; } = string.Empty;
 
     [JsonPropertyName("name")]
-    public string Name { get; set; }
+    public string Name { get; set; } = string.Empty;
 
     [JsonPropertyName("description")]
-    public string Description { get; set; }
+    public string? Description { get; set; }
 
     [JsonPropertyName("parentProjectId")]
-    public string ParentProjectId { get; set; }
+    public string? ParentProjectId { get; set; }
 
     [JsonPropertyName("ownerId")]
-    public string OwnerId { get; set; }
-}
+    public string? OwnerId { get; set; }
+}

src/TableauSharp/Projects/Services/ProjectService.cs

@@ -31,9 +31,9 @@ public async Task<IEnumerable<TableauProject>> GetAllAsync(CancellationToken can
         {
             projects.Add(new TableauProject
             {
-                Id = p.GetProperty("id").GetString(),
-                Name = p.GetProperty("name").GetString(),
-                Description = p.GetProperty("description").GetString(),
+                Id = p.GetProperty("id").GetString() ?? string.Empty,
+                Name = p.GetProperty("name").GetString() ?? string.Empty,
+                Description = p.TryGetProperty("description", out var description) ? description.GetString() : null,
                 ParentProjectId = p.TryGetProperty("parentProjectId", out var parent) ? parent.GetString() : null,
                 OwnerId = p.GetProperty("owner").GetProperty("id").GetString()
             });
@@ -55,9 +55,9 @@ public async Task<TableauProject> GetByIdAsync(string projectId, CancellationTok
 
         return new TableauProject
         {
-            Id = p.GetProperty("id").GetString(),
-            Name = p.GetProperty("name").GetString(),
-            Description = p.GetProperty("description").GetString(),
+            Id = p.GetProperty("id").GetString() ?? string.Empty,
+            Name = p.GetProperty("name").GetString() ?? string.Empty,
+            Description = p.TryGetProperty("description", out var description) ? description.GetString() : null,
             ParentProjectId = p.TryGetProperty("parentProjectId", out var parent) ? parent.GetString() : null,
             OwnerId = p.GetProperty("owner").GetProperty("id").GetString()
         };
@@ -80,9 +80,9 @@ public async Task<TableauProject> CreateAsync(ProjectCreateRequest request, Canc
 
         return new TableauProject
         {
-            Id = p.GetProperty("id").GetString(),
-            Name = p.GetProperty("name").GetString(),
-            Description = p.GetProperty("description").GetString(),
+            Id = p.GetProperty("id").GetString() ?? string.Empty,
+            Name = p.GetProperty("name").GetString() ?? string.Empty,
+            Description = p.TryGetProperty("description", out var description) ? description.GetString() : null,
             ParentProjectId = p.TryGetProperty("parentProjectId", out var parent) ? parent.GetString() : null,
             OwnerId = p.GetProperty("owner").GetProperty("id").GetString()
         };
@@ -105,9 +105,9 @@ public async Task<TableauProject> UpdateAsync(string projectId, ProjectUpdateReq
 
         return new TableauProject
         {
-            Id = p.GetProperty("id").GetString(),
-            Name = p.GetProperty("name").GetString(),
-            Description = p.GetProperty("description").GetString(),
+            Id = p.GetProperty("id").GetString() ?? string.Empty,
+            Name = p.GetProperty("name").GetString() ?? string.Empty,
+            Description = p.TryGetProperty("description", out var description) ? description.GetString() : null,
             ParentProjectId = p.TryGetProperty("parentProjectId", out var parent) ? parent.GetString() : null,
             OwnerId = p.GetProperty("owner").GetProperty("id").GetString()
         };

src/TableauSharp/TableauSharp.csproj

@@ -4,6 +4,7 @@
     <TargetFramework>net8.0</TargetFramework>
     <ImplicitUsings>enable</ImplicitUsings>
     <Nullable>enable</Nullable>
+    <TreatWarningsAsErrors>true</TreatWarningsAsErrors>
 
     <!-- NuGet package metadata -->
     <PackageId>TableauSharp</PackageId>

src/TableauSharp/Users/Models/GroupCreateRequest.cs

@@ -2,5 +2,5 @@ namespace TableauSharp.Users.Models;
 
 public class GroupCreateRequest
 {
-    public string Name { get; set; }
-}
+    public string Name { get; set; } = string.Empty;
+}

src/TableauSharp/Users/Models/TableauGroup.cs

@@ -5,11 +5,11 @@ namespace TableauSharp.Users.Models;
 public class TableauGroup
 {
     [JsonPropertyName("id")]
-    public string Id { get; set; }
+    public string Id { get; set; } = string.Empty;
 
     [JsonPropertyName("name")]
-    public string Name { get; set; }
+    public string Name { get; set; } = string.Empty;
 
     [JsonPropertyName("users")]
     public List<TableauUser> Users { get; set; } = new();
-}
+}

src/TableauSharp/Users/Models/TableauUser.cs

@@ -5,20 +5,20 @@ namespace TableauSharp.Users.Models;
 public class TableauUser
 {
     [JsonPropertyName("id")]
-    public string Id { get; set; }
+    public string Id { get; set; } = string.Empty;
 
     [JsonPropertyName("name")]
-    public string Name { get; set; }
+    public string Name { get; set; } = string.Empty;
 
     [JsonPropertyName("fullName")]
-    public string FullName { get; set; }
+    public string? FullName { get; set; }
 
     [JsonPropertyName("email")]
-    public string Email { get; set; }
+    public string? Email { get; set; }
 
     [JsonPropertyName("siteRole")]
-    public string SiteRole { get; set; }
+    public string SiteRole { get; set; } = string.Empty;
 
     [JsonPropertyName("isActive")]
     public bool IsActive { get; set; }
-}
+}

src/TableauSharp/Users/Models/UserCreateRequest.cs

@@ -2,7 +2,7 @@ namespace TableauSharp.Users.Models;
 
 public class UserCreateRequest
 {
-    public string Name { get; set; }
-    public string Email { get; set; }
-    public string SiteRole { get; set; }
-}
+    public string Name { get; set; } = string.Empty;
+    public string? Email { get; set; }
+    public string SiteRole { get; set; } = string.Empty;
+}

src/TableauSharp/Users/Models/UserUpdateRequest.cs

@@ -2,6 +2,6 @@ namespace TableauSharp.Users.Models;
 
 public class UserUpdateRequest
 {
-    public string SiteRole { get; set; }
+    public string SiteRole { get; set; } = string.Empty;
     public bool IsActive { get; set; }
-}
+}

src/TableauSharp/Users/Services/GroupService.cs

@@ -31,8 +31,8 @@ public async Task<IEnumerable<TableauGroup>> GetAllAsync(CancellationToken cance
         {
             groups.Add(new TableauGroup
             {
-                Id = g.GetProperty("id").GetString(),
-                Name = g.GetProperty("name").GetString()
+                Id = g.GetProperty("id").GetString() ?? string.Empty,
+                Name = g.GetProperty("name").GetString() ?? string.Empty
             });
         }
 
@@ -52,8 +52,8 @@ public async Task<TableauGroup> GetByIdAsync(string groupId, CancellationToken c
 
         return new TableauGroup
         {
-            Id = g.GetProperty("id").GetString(),
-            Name = g.GetProperty("name").GetString()
+            Id = g.GetProperty("id").GetString() ?? string.Empty,
+            Name = g.GetProperty("name").GetString() ?? string.Empty
         };
     }
 
@@ -74,8 +74,8 @@ public async Task<TableauGroup> CreateAsync(GroupCreateRequest request, Cancella
 
         return new TableauGroup
         {
-            Id = g.GetProperty("id").GetString(),
-            Name = g.GetProperty("name").GetString()
+            Id = g.GetProperty("id").GetString() ?? string.Empty,
+            Name = g.GetProperty("name").GetString() ?? string.Empty
         };
     }
 

src/TableauSharp/Users/Services/UserService.cs

@@ -31,10 +31,10 @@ public async Task<IEnumerable<TableauUser>> GetAllAsync(CancellationToken cancel
         {
             users.Add(new TableauUser
             {
-                Id = u.GetProperty("id").GetString(),
-                Name = u.GetProperty("name").GetString(),
-                Email = u.GetProperty("email").GetString(),
-                SiteRole = u.GetProperty("siteRole").GetString()
+                Id = u.GetProperty("id").GetString() ?? string.Empty,
+                Name = u.GetProperty("name").GetString() ?? string.Empty,
+                Email = u.TryGetProperty("email", out var email) ? email.GetString() : null,
+                SiteRole = u.GetProperty("siteRole").GetString() ?? string.Empty
             });
         }
 
@@ -54,10 +54,10 @@ public async Task<TableauUser> GetByIdAsync(string userId, CancellationToken can
 
         return new TableauUser
         {
-            Id = u.GetProperty("id").GetString(),
-            Name = u.GetProperty("name").GetString(),
-            Email = u.GetProperty("email").GetString(),
-            SiteRole = u.GetProperty("siteRole").GetString()
+            Id = u.GetProperty("id").GetString() ?? string.Empty,
+            Name = u.GetProperty("name").GetString() ?? string.Empty,
+            Email = u.TryGetProperty("email", out var email) ? email.GetString() : null,
+            SiteRole = u.GetProperty("siteRole").GetString() ?? string.Empty
         };
     }
 
@@ -78,10 +78,10 @@ public async Task<TableauUser> CreateAsync(UserCreateRequest request, Cancellati
 
         return new TableauUser
         {
-            Id = u.GetProperty("id").GetString(),
-            Name = u.GetProperty("name").GetString(),
-            Email = u.GetProperty("email").GetString(),
-            SiteRole = u.GetProperty("siteRole").GetString()
+            Id = u.GetProperty("id").GetString() ?? string.Empty,
+            Name = u.GetProperty("name").GetString() ?? string.Empty,
+            Email = u.TryGetProperty("email", out var email) ? email.GetString() : null,
+            SiteRole = u.GetProperty("siteRole").GetString() ?? string.Empty
         };
     }
 
@@ -102,10 +102,10 @@ public async Task<TableauUser> UpdateAsync(string userId, UserUpdateRequest requ
 
         return new TableauUser
         {
-            Id = u.GetProperty("id").GetString(),
-            Name = u.GetProperty("name").GetString(),
-            Email = u.GetProperty("email").GetString(),
-            SiteRole = u.GetProperty("siteRole").GetString()
+            Id = u.GetProperty("id").GetString() ?? string.Empty,
+            Name = u.GetProperty("name").GetString() ?? string.Empty,
+            Email = u.TryGetProperty("email", out var email) ? email.GetString() : null,
+            SiteRole = u.GetProperty("siteRole").GetString() ?? string.Empty
         };
     }
 

src/TableauSharp/Workbooks/Models/ExportRequest.cs

@@ -2,9 +2,9 @@ namespace TableauSharp.Workbooks.Models;
 
 public class ExportRequest
 {
-    public string WorkbookId { get; set; }
-    public string ViewId { get; set; }
-    public string Format { get; set; } // "PDF", "PNG", "CSV"
+    public string? WorkbookId { get; set; }
+    public string ViewId { get; set; } = string.Empty;
+    public string Format { get; set; } = string.Empty; // "PDF", "PNG", "CSV"
     public int? MaxWidth { get; set; }
     public int? MaxHeight { get; set; }
-}
+}

src/TableauSharp/Workbooks/Models/ExportResponse.cs

@@ -2,7 +2,7 @@ namespace TableauSharp.Workbooks.Models;
 
 public class ExportResponse
 {
-    public byte[] FileContent { get; set; }
-    public string ContentType { get; set; }
-    public string FileName { get; set; }
-}
+    public byte[] FileContent { get; set; } = [];
+    public string ContentType { get; set; } = string.Empty;
+    public string FileName { get; set; } = string.Empty;
+}