From 8144f271a4f70bdf8771f87612a24ac6a89f845f Mon Sep 17 00:00:00 2001 From: Jeremy Caney Date: Wed, 8 Jul 2026 15:12:31 -0700 Subject: [PATCH] Disable mapping on new `Province` field The recently introduced `Province` field (789d3d6b) is optional, and only supported on the Demo and Trial Request forms. But the underlying `Contact` is used elsewhere. That's fine unless you try to (re)use the binding model as a mapping model with the `ReverseTopicMappingService`, in which case it will fail to find the corresponding value. The fix? Disable mapping for this field. Long-term we need to rethink this structure. For now, however, this is a sufficient solution. --- Areas/Forms/Models/Partials/Contact.cs | 1 + 1 file changed, 1 insertion(+) diff --git a/Areas/Forms/Models/Partials/Contact.cs b/Areas/Forms/Models/Partials/Contact.cs index 6a54c666..e29406b9 100644 --- a/Areas/Forms/Models/Partials/Contact.cs +++ b/Areas/Forms/Models/Partials/Contact.cs @@ -29,6 +29,7 @@ public record Contact : CoreContact { /// required. It is included here to provide support for the and forms. /// + [DisableMapping] [StringLength(255)] [Display(Name="State/Province")] public string Province { get; set; }