mypy and formatting

lnbits/extensions/admin/crud.py

@@ -1,4 +1,4 @@
-from typing import List
+from typing import List, Optional
 
 from lnbits.core.crud import create_payment
 from lnbits.helpers import urlsafe_short_hash
@@ -9,7 +9,7 @@ from . import db
 from .models import AdminSettings, UpdateSettings
 
 
-async def update_wallet_balance(wallet_id: str, amount: int) -> str:
+async def update_wallet_balance(wallet_id: str, amount: int) -> None:
     temp_id = f"temp_{urlsafe_short_hash()}"
     internal_id = f"internal_{urlsafe_short_hash()}"
 
@@ -36,7 +36,7 @@ async def get_settings() -> AdminSettings:
     return settings
 
 
-async def update_settings(data: UpdateSettings) -> Settings:
+async def update_settings(data: UpdateSettings) -> Optional[Settings]:
     fields = []
     for key, value in data.dict(exclude_none=True).items():
         if not key in read_only_variables:

lnbits/extensions/admin/views.py

@@ -17,7 +17,7 @@ templates = Jinja2Templates(directory="templates")
 
 
 @admin_ext.get("/", response_class=HTMLResponse)
-async def index(request: Request, user: User = Depends(check_admin)):
+async def index(request: Request, user: User = Depends(check_admin)):  # type: ignore
     WALLET = get_wallet_class()
     error, balance = await WALLET.status()
 

lnbits/extensions/admin/views_api.py

@@ -1,6 +1,7 @@
 from http import HTTPStatus
 
-from fastapi import Body, Depends, Query
+from fastapi import Body
+from fastapi.params import Depends
 from starlette.exceptions import HTTPException
 
 from lnbits.core.crud import get_wallet
@@ -51,7 +52,8 @@ async def api_update_settings(
     data: UpdateSettings = Body(...),
 ):
     settings = await update_settings(data)
-    return {"status": "Success", "settings": settings.dict()}
+    if settings:
+        return {"status": "Success", "settings": settings.dict()}
 
 
 @admin_ext.delete(

lnbits/server.py

@@ -19,7 +19,11 @@ from lnbits.settings import set_cli_settings, settings
 )
 @click.option("--port", default=settings.port, help="Port to listen on")
 @click.option("--host", default=settings.host, help="Host to run LNBits on")
-@click.option("--forwarded-allow-ips", default=settings.forwarded_allow_ips, help="Allowed proxy servers")
+@click.option(
+    "--forwarded-allow-ips",
+    default=settings.forwarded_allow_ips,
+    help="Allowed proxy servers",
+)
 @click.option("--ssl-keyfile", default=None, help="Path to SSL keyfile")
 @click.option("--ssl-certfile", default=None, help="Path to SSL certificate")
 @click.pass_context