chore: remove setUserConfig as unused

Author: Bilb

include/user_config.hpp

@@ -26,7 +26,6 @@ class UserConfigWrapper : public ConfigBaseImpl, public Napi::ObjectWrap<UserCon
     void setName(const Napi::CallbackInfo& info);
     void setNameTruncated(const Napi::CallbackInfo& info);
     void setNewProfilePic(const Napi::CallbackInfo& info);
-    void setUserConfig(const Napi::CallbackInfo& info);
 
     void setReuploadProfilePic(const Napi::CallbackInfo& info);
     Napi::Value getProfileUpdatedSeconds(const Napi::CallbackInfo& info);

src/user_config.cpp

@@ -30,7 +30,6 @@ void UserConfigWrapper::Init(Napi::Env env, Napi::Object exports) {
                     InstanceMethod("setPriority", &UserConfigWrapper::setPriority),
                     InstanceMethod("setName", &UserConfigWrapper::setName),
                     InstanceMethod("setNameTruncated", &UserConfigWrapper::setNameTruncated),
-                    InstanceMethod("setUserConfig", &UserConfigWrapper::setUserConfig),
                     InstanceMethod("setNewProfilePic", &UserConfigWrapper::setNewProfilePic),
                     InstanceMethod(
                             "getEnableBlindedMsgRequest",
@@ -77,61 +76,6 @@ Napi::Value UserConfigWrapper::getProfilePic(const Napi::CallbackInfo& info) {
     });
 }
 
-void UserConfigWrapper::setUserConfig(const Napi::CallbackInfo& info) {
-    return wrapExceptions(info, [&] {
-        auto env = info.Env();
-        assertInfoLength(info, 1);
-        auto configObj = info[0];
-        assertIsObject(configObj);
-
-        auto obj = configObj.As<Napi::Object>();
-        if (obj.IsEmpty()) {
-            return;
-        }
-        // Handle priority field
-        if (auto priority = maybeNonemptyInt(
-                    obj.Get("priority"), "UserConfigWrapper::setUserConfig - priority")) {
-            auto new_priority = toPriority(priority.value(), config.get_nts_priority());
-            config.set_nts_priority(new_priority);
-        }
-
-        // Handle nameTruncated field
-        if (auto new_name = maybeNonemptyString(
-                    obj.Get("name"), "UserConfigWrapper::setUserConfig - name")) {
-            config.set_name_truncated(*new_name);  // truncates silently if too long
-        }
-
-        // Handle newProfilePic field
-        if (auto newProfilePic = maybeNonemptyProfilePic(
-                    obj.Get("newProfilePic"), "UserConfigWrapper::setUserConfig - newProfilePic")) {
-            config.set_profile_pic(*newProfilePic);
-        }
-
-        // Handle reuploadProfilePic field
-        if (auto reuploadProfilePic = maybeNonemptyProfilePic(
-                    obj.Get("reuploadProfilePic"),
-                    "UserConfigWrapper::setUserConfig - reuploadProfilePic")) {
-            config.set_reupload_profile_pic(*reuploadProfilePic);
-        }
-
-        // Handle enableBlindedMsgRequest field
-        if (auto blindedMsgReqCpp = maybeNonemptyBoolean(
-                    obj.Get("enableBlindedMsgRequest"),
-                    "UserConfigWrapper::setUserConfig - enableBlindedMsgRequest")) {
-            config.set_blinded_msgreqs(*blindedMsgReqCpp);
-        }
-
-        // Handle noteToSelfExpiry field
-        if (auto new_nts_expiry_seconds = maybeNonemptyInt(
-                    obj.Get("noteToSelfExpirySeconds"),
-                    "UserConfigWrapper::setUserConfig - noteToSelfExpiry")) {
-            auto expiry = obj.Get("noteToSelfExpiry");
-
-            config.set_nts_expiry(std::chrono::seconds{*new_nts_expiry_seconds});
-        }
-    });
-}
-
 void UserConfigWrapper::setPriority(const Napi::CallbackInfo& info) {
     return wrapExceptions(info, [&] {
         auto env = info.Env();

types/user/userconfig.d.ts

@@ -17,24 +17,7 @@ declare module 'libsession_util_nodejs' {
     setPriority: (priority: number) => void;
     setName: (name: string) => void;
     setNameTruncated: (name: string) => void;
-    /**
-     *
-     * Batch set user config fields.
-     * Set a field to null to not change it
-     * for example,
-     * - set newProfilePic to null to not change the profile picture
-     * - set newProfilePic to url: null, key: null to erase it
-     *
-     * `name` set to null means to not change it, but `name` set to `''` means to erase it
-     */
-    setUserConfig: (details: {
-      priority: number | null;
-      name: string | null;
-      newProfilePic: ProfilePicture | null;
-      reuploadProfilePic: ProfilePicture | null;
-      enableBlindedMsgRequest: boolean | null;
-      noteToSelfExpiry: number | null;
-    }) => void;
+
     /**
      * Call this when uploading a new profile picture (i.e. not an auto reupload)
      */
@@ -77,7 +60,6 @@ declare module 'libsession_util_nodejs' {
     public setNameTruncated: UserConfigWrapper['setNameTruncated'];
     public setNewProfilePic: UserConfigWrapper['setNewProfilePic'];
     public setReuploadProfilePic: UserConfigWrapper['setReuploadProfilePic'];
-    public setUserConfig: UserConfigWrapper['setUserConfig'];
     public getProfileUpdatedSeconds: UserConfigWrapper['getProfileUpdatedSeconds'];
     public getProfilePicWithKeyHex: UserConfigWrapper['getProfilePicWithKeyHex'];
     public getEnableBlindedMsgRequest: UserConfigWrapper['getEnableBlindedMsgRequest'];
@@ -102,7 +84,6 @@ declare module 'libsession_util_nodejs' {
     | MakeActionCall<UserConfigWrapper, 'setNameTruncated'>
     | MakeActionCall<UserConfigWrapper, 'setNewProfilePic'>
     | MakeActionCall<UserConfigWrapper, 'setReuploadProfilePic'>
-    | MakeActionCall<UserConfigWrapper, 'setUserConfig'>
     | MakeActionCall<UserConfigWrapper, 'getProfileUpdatedSeconds'>
     | MakeActionCall<UserConfigWrapper, 'getProfilePicWithKeyHex'>
     | MakeActionCall<UserConfigWrapper, 'getEnableBlindedMsgRequest'>