Remove deprecated properties

BTCPayServer.Client/Models/ApplicationUserData.cs

@@ -1,5 +1,7 @@
 using System;
+using System.Collections.Generic;
 using Newtonsoft.Json;
+using Newtonsoft.Json.Linq;
 
 namespace BTCPayServer.Client.Models
 {
@@ -62,5 +64,8 @@ namespace BTCPayServer.Client.Models
         public DateTimeOffset? Created { get; set; }
 
         public bool Disabled { get; set; }
+
+        [JsonExtensionData]
+        public IDictionary<string, JToken> AdditionalData { get; set; } = new Dictionary<string, JToken>();
     }
 }

BTCPayServer.Client/Models/StoreData.cs

@@ -14,22 +14,10 @@ namespace BTCPayServer.Client.Models
 
     public class StoreUserData : ApplicationUserData
     {
-        /// <summary>
-        /// the id of the user
-        /// </summary>
-        [Obsolete("Use Id instead")]
-        public string UserId { get; set; }
-
         /// <summary>
         /// the store role of the user
         /// </summary>
         public string StoreRole { get; set; }
-
-        /// <summary>
-        /// the store role of the user
-        /// </summary>
-        [Obsolete("Use StoreRole instead")]
-        public string Role { get; set; }
     }
 
     public class RoleData

BTCPayServer.Tests/GreenfieldAPITests.cs

@@ -4018,7 +4018,9 @@ namespace BTCPayServer.Tests
             var users = await client.GetStoreUsers(user.StoreId);
             var storeUser = Assert.Single(users);
             Assert.Equal(user.UserId, storeUser.Id);
+            Assert.Equal(user.UserId, storeUser.AdditionalData["userId"].ToString());
             Assert.Equal(ownerRole.Id, storeUser.StoreRole);
+            Assert.Equal(ownerRole.Id, storeUser.AdditionalData["role"].ToString());
             Assert.Equal(user.Email, storeUser.Email);
             Assert.Equal("The Admin", storeUser.Name);
             Assert.Equal("avatar.jpg", storeUser.ImageUrl);

BTCPayServer/Controllers/GreenField/GreenfieldStoreUsersController.cs

@@ -14,6 +14,7 @@ using Microsoft.AspNetCore.Authorization;
 using Microsoft.AspNetCore.Cors;
 using Microsoft.AspNetCore.Identity;
 using Microsoft.AspNetCore.Mvc;
+using Newtonsoft.Json.Linq;
 using NicolasDorier.RateLimits;
 using StoreData = BTCPayServer.Data.StoreData;
 
@@ -46,7 +47,7 @@ namespace BTCPayServer.Controllers.Greenfield
         public async Task<IActionResult> GetStoreUsers()
         {
             var store = HttpContext.GetStoreData();
-            return store == null ? StoreNotFound() : Ok(await FromModel(store));
+            return store == null ? StoreNotFound() : Ok(await ToAPI(store));
         }
 
         [Authorize(Policy = Policies.CanModifyStoreSettings, AuthenticationSchemes = AuthenticationSchemes.Greenfield)]
@@ -74,10 +75,11 @@ namespace BTCPayServer.Controllers.Greenfield
             var store = HttpContext.GetStoreData();
             if (store == null)
                 return StoreNotFound();
-#pragma warning disable CS0618 // Type or member is obsolete
+
-            request.StoreRole ??= request.Role;
+            // Deprecated properties
-            request.Id ??= request.UserId;
+            request.StoreRole ??= request.AdditionalData.TryGetValue("role", out var role) ? role.ToString() : null;
-#pragma warning restore CS0618 // Type or member is obsolete
+            request.Id ??= request.AdditionalData.TryGetValue("userId", out var userId) ? userId.ToString() : null;
+            /////
 
             var user = await _userManager.FindByIdOrEmail(idOrEmail ?? request.Id);
             if (user == null)
@@ -102,7 +104,7 @@ namespace BTCPayServer.Controllers.Greenfield
                 : this.CreateAPIError(409, "duplicate-store-user-role", "The user is already added to the store");
         }
 
-        private async Task<IEnumerable<StoreUserData>> FromModel(StoreData store)
+        private async Task<IEnumerable<StoreUserData>> ToAPI(StoreData store)
         {
             var storeUsers = new List<StoreUserData>();
             foreach (var storeUser in store.UserStores)
@@ -112,10 +114,12 @@ namespace BTCPayServer.Controllers.Greenfield
                     continue;
                 var data = await UserService.ForAPI<StoreUserData>(user, [], _callbackGenerator, _uriResolver, Request);
                 data.StoreRole = storeUser.StoreRoleId;
-#pragma warning disable CS0618 // Type or member is obsolete
+
-                data.UserId = storeUser.ApplicationUserId;
+                // Deprecated properties
-                data.Role = storeUser.StoreRoleId;
+                data.AdditionalData["userId"] = new JValue(storeUser.ApplicationUserId);
-#pragma warning restore CS0618 // Type or member is obsolete
+                data.AdditionalData["role"] = new JValue(storeUser.StoreRoleId);
+                /////
+
                 storeUsers.Add(data);
             }
             return storeUsers;