Address further feedback, correct nonnull Halodto.

Author: Baaaaaz

messages/src/main/proto/data_transfer_objects.proto

@@ -101,8 +101,8 @@ message LightDto {
 }
 
 message HaloDto {
-  google.protobuf.StringValue id = 1;
-  google.protobuf.StringValue name = 2;
+  string id = 1;
+  string name = 2;
   bool gm_only = 3;
   bool owner_only = 4;
   bool inner = 5;

src/main/java/net/rptools/maptool/model/Halo.java

@@ -15,7 +15,6 @@
 package net.rptools.maptool.model;
 
 import com.google.common.collect.ImmutableList;
-import com.google.protobuf.StringValue;
 import java.io.Serial;
 import java.io.Serializable;
 import java.util.*;
@@ -173,8 +172,8 @@ public String toString() {
 
   public static @Nonnull Halo fromDto(@Nonnull HaloDto dto) {
     return new Halo(
-        GUID.valueOf(dto.getId().getValue()),
-        dto.getName().getValue(),
+        GUID.valueOf(dto.getId()),
+        dto.getName(),
         dto.getGmOnly(),
         dto.getOwnerOnly(),
         dto.getInner(),
@@ -188,8 +187,8 @@ public String toString() {
 
   public @Nonnull HaloDto toDto() {
     var dto = HaloDto.newBuilder();
-    dto.setId(StringValue.of(id.toString()));
-    dto.setName(StringValue.of(name));
+    dto.setId(id.toString());
+    dto.setName(name);
     dto.setGmOnly(gmOnly);
     dto.setOwnerOnly(ownerOnly);
     dto.setInner(inner);