cleanup and typing fix for data

lnbits/extensions/admin/models.py

@@ -1,13 +1,15 @@
+from typing import List
+
 from fastapi import Query
 from pydantic import BaseModel
 
 
 class UpdateSettings(BaseModel):
     lnbits_backend_wallet_class: str = Query(None)
-    lnbits_admin_users: str = Query(None) #this should be List[str] ??
-    lnbits_allowed_users: str = Query(None) #this should be List[str] ??
-    lnbits_admin_ext: str = Query(None) #this should be List[str] ??
-    lnbits_disabled_ext: str = Query(None) #this should be List[str] ??
+    lnbits_admin_users: List[str] = Query(None) 
+    lnbits_allowed_users: List[str] = Query(None) 
+    lnbits_admin_ext: List[str] = Query(None) 
+    lnbits_disabled_ext: List[str] = Query(None) 
     lnbits_funding_source: str = Query(None)
     lnbits_force_https: bool = Query(None)
     lnbits_reserve_fee_min: int = Query(None, ge=0)
@@ -21,4 +23,4 @@ class UpdateSettings(BaseModel):
     lnbits_denomination: str = Query(None)
     lnbits_theme: str = Query(None)
     lnbits_custom_logo: str = Query(None)
-    lnbits_ad_space: str = Query(None) #this should be List[str] ??
+    lnbits_ad_space: List[str] = Query(None) 

lnbits/extensions/admin/templates/admin/index.html

@@ -210,25 +210,7 @@
       },
       updateSettings() {
         let data = {          
-          lnbits_backend_wallet_class: this.formData.lnbits_backend_wallet_class,
-          lnbits_admin_users: this.formData.lnbits_admin_users.toString(),
-          lnbits_allowed_users: this.formData.lnbits_allowed_users.toString(),
-          lnbits_admin_ext: this.formData.lnbits_admin_ext,
-          lnbits_disabled_ext: this.formData.lnbits_disabled_ext,
-          lnbits_funding_source: this.formData.lnbits_funding_source,
-          lnbits_force_https: this.formData.lnbits_force_https,
-          lnbits_reserve_fee_min: this.formData.lnbits_reserve_fee_min,
-          lnbits_reserve_fee_percent: this.formData.lnbits_reserve_fee_percent,
-          lnbits_service_fee: this.formData.lnbits_service_fee,
-          lnbits_hide_api: this.formData.lnbits_hide_api,
-          lnbits_site_title: this.formData.lnbits_site_title,
-          lnbits_site_tagline: this.formData.lnbits_site_tagline,
-          lnbits_site_description: this.formData.lnbits_site_description,
-          lnbits_default_wallet_name: this.formData.lnbits_default_wallet_name,
-          lnbits_denomination: this.formData.lnbits_denomination,
-          lnbits_theme: this.formData.lnbits_theme,
-          lnbits_custom_logo: this.formData.lnbits_custom_logo,
-          lnbits_ad_space: this.formData.lnbits_ad_space.toString()
+          ...this.formData
         }
         LNbits.api
           .request(

lnbits/extensions/admin/views_api.py

@@ -44,7 +44,6 @@ async def api_update_settings(
     data: UpdateSettings = Body(...),
 ):
     settings = await update_settings(data)
-    logger.debug(settings)
     return {"status": "Success", "settings": settings.dict()}