Merge branch 'RoboSats:main' into add-location-to-f2f-frontend

.env-sample

@@ -91,11 +91,6 @@ MAKER_FEE_SPLIT=0.125
 # Leaving the default value (20%) will grant the DevFund contributor badge.
 DEVFUND = 0.2
 
-# Bond size as percentage (%)
-DEFAULT_BOND_SIZE = 3
-MIN_BOND_SIZE = 1
-MAX_BOND_SIZE = 15
-
 # Time out penalty for canceling takers in SECONDS
 PENALTY_TIMEOUT = 60
 # Time between routing attempts of buyer invoice in MINUTES
@@ -109,9 +104,11 @@ DISABLE_ORDER_LOGS = False
 # Coordinator activity limits
 MAX_PUBLIC_ORDERS = 100
 
-# Trade limits in satoshis
-MIN_TRADE = 20000
-MAX_TRADE = 5000000
+# Coordinator Order size limits in Satoshi
+# Minimum order size (must be bigger than DB constrain in /robosats/settings.py MIN_TRADE, currently 20_000 Sats)
+MIN_ORDER_SIZE = 20000
+# Minimum order size (must be smaller than DB constrain in /robosats/settings.py MAX_TRADE, currently 5_000_000 Sats)
+MAX_ORDER_SIZE = 5000000
 
 # For CLTV_expiry calculation
 # Assume 8 min/block assumed
@@ -123,16 +120,6 @@ MAX_MINING_NETWORK_SPEEDUP_EXPECTED = 1.7
 EXP_MAKER_BOND_INVOICE = 300
 EXP_TAKER_BOND_INVOICE = 200
 
-# Time a order is public in the book HOURS
-DEFAULT_PUBLIC_ORDER_DURATION = 24
-MAX_PUBLIC_ORDER_DURATION = 24
-MIN_PUBLIC_ORDER_DURATION = 0.166
-
-# Default time to provide a valid invoice and the trade escrow MINUTES
-INVOICE_AND_ESCROW_DURATION = 180
-# Time to confim chat and confirm fiat (time to Fiat Sent confirmation) HOURS
-FIAT_EXCHANGE_DURATION = 24
-
 # ROUTING
 # Proportional routing fee limit (fraction of total payout: % / 100)
 PROPORTIONAL_ROUTING_FEE_LIMIT = 0.001

Dockerfile

@@ -1,4 +1,4 @@
-FROM python:3.11.4-slim-bullseye
+FROM python:3.11.6-slim-bullseye
 ARG DEBIAN_FRONTEND=noninteractive
 
 RUN mkdir -p /usr/src/robosats

api/logics.py

@@ -18,8 +18,8 @@
 ESCROW_USERNAME = config("ESCROW_USERNAME")
 PENALTY_TIMEOUT = int(config("PENALTY_TIMEOUT"))
 
-MIN_TRADE = int(config("MIN_TRADE"))
-MAX_TRADE = int(config("MAX_TRADE"))
+MIN_ORDER_SIZE = config("MIN_ORDER_SIZE", cast=int, default=20_000)
+MAX_ORDER_SIZE = config("MAX_ORDER_SIZE", cast=int, default=5_000_000)
 
 EXP_MAKER_BOND_INVOICE = int(config("EXP_MAKER_BOND_INVOICE"))
 EXP_TAKER_BOND_INVOICE = int(config("EXP_TAKER_BOND_INVOICE"))
@@ -29,9 +29,6 @@
     config("MAX_MINING_NETWORK_SPEEDUP_EXPECTED")
 )
 
-INVOICE_AND_ESCROW_DURATION = int(config("INVOICE_AND_ESCROW_DURATION"))
-FIAT_EXCHANGE_DURATION = int(config("FIAT_EXCHANGE_DURATION"))
-
 
 class Logics:
     @classmethod
@@ -90,20 +87,20 @@ def validate_already_maker_or_taker(cls, user):
     def validate_order_size(cls, order):
         """Validates if order size in Sats is within limits at t0"""
         if not order.has_range:
-            if order.t0_satoshis > MAX_TRADE:
+            if order.t0_satoshis > MAX_ORDER_SIZE:
                 return False, {
                     "bad_request": "Your order is too big. It is worth "
                     + "{:,}".format(order.t0_satoshis)
                     + " Sats now, but the limit is "
-                    + "{:,}".format(MAX_TRADE)
+                    + "{:,}".format(MAX_ORDER_SIZE)
                     + " Sats"
                 }
-            if order.t0_satoshis < MIN_TRADE:
+            if order.t0_satoshis < MIN_ORDER_SIZE:
                 return False, {
                     "bad_request": "Your order is too small. It is worth "
                     + "{:,}".format(order.t0_satoshis)
                     + " Sats now, but the limit is "
-                    + "{:,}".format(MIN_TRADE)
+                    + "{:,}".format(MIN_ORDER_SIZE)
                     + " Sats"
                 }
         elif order.has_range:
@@ -117,20 +114,20 @@ def validate_order_size(cls, order):
                 return False, {
                     "bad_request": "Maximum range amount must be at least 50 percent higher than the minimum amount"
                 }
-            elif max_sats > MAX_TRADE:
+            elif max_sats > MAX_ORDER_SIZE:
                 return False, {
                     "bad_request": "Your order maximum amount is too big. It is worth "
                     + "{:,}".format(int(max_sats))
                     + " Sats now, but the limit is "
-                    + "{:,}".format(MAX_TRADE)
+                    + "{:,}".format(MAX_ORDER_SIZE)
                     + " Sats"
                 }
-            elif min_sats < MIN_TRADE:
+            elif min_sats < MIN_ORDER_SIZE:
                 return False, {
                     "bad_request": "Your order minimum amount is too small. It is worth "
                     + "{:,}".format(int(min_sats))
                     + " Sats now, but the limit is "
-                    + "{:,}".format(MIN_TRADE)
+                    + "{:,}".format(MIN_ORDER_SIZE)
                     + " Sats"
                 }
             elif min_sats < max_sats / 15:
@@ -590,7 +587,7 @@ def compute_swap_fee_rate(balance):
         shape = str(config("SWAP_FEE_SHAPE"))
 
         if shape == "linear":
-            MIN_SWAP_FEE = float(config("MIN_SWAP_FEE"))
+            MIN_SWAP_FEE = config("MIN_SWAP_FEE", cast=float, default=0.01)
             MIN_POINT = float(config("MIN_POINT"))
             MAX_SWAP_FEE = float(config("MAX_SWAP_FEE"))
             MAX_POINT = float(config("MAX_POINT"))
@@ -603,7 +600,7 @@ def compute_swap_fee_rate(balance):
                 )
 
         elif shape == "exponential":
-            MIN_SWAP_FEE = float(config("MIN_SWAP_FEE"))
+            MIN_SWAP_FEE = config("MIN_SWAP_FEE", cast=float, default=0.01)
             MAX_SWAP_FEE = float(config("MAX_SWAP_FEE"))
             SWAP_LAMBDA = float(config("SWAP_LAMBDA"))
             swap_fee_rate = MIN_SWAP_FEE + (MAX_SWAP_FEE - MIN_SWAP_FEE) * math.exp(

api/migrations/0043_order_latitude_order_longitude.py

@@ -0,0 +1,39 @@
+# Generated by Django 4.2.5 on 2023-10-03 20:12
+
+import django.core.validators
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("api", "0042_alter_order_logs_alter_robot_avatar"),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name="order",
+            name="latitude",
+            field=models.DecimalField(
+                decimal_places=6,
+                max_digits=8,
+                null=True,
+                validators=[
+                    django.core.validators.MinValueValidator(-90),
+                    django.core.validators.MaxValueValidator(90),
+                ],
+            ),
+        ),
+        migrations.AddField(
+            model_name="order",
+            name="longitude",
+            field=models.DecimalField(
+                decimal_places=6,
+                max_digits=9,
+                null=True,
+                validators=[
+                    django.core.validators.MinValueValidator(-180),
+                    django.core.validators.MaxValueValidator(180),
+                ],
+            ),
+        ),
+    ]

api/migrations/0044_alter_markettick_fee_and_more.py

@@ -0,0 +1,53 @@
+# Generated by Django 4.2.6 on 2023-10-11 10:48
+
+import django.core.validators
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("api", "0043_order_latitude_order_longitude"),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name="markettick",
+            name="fee",
+            field=models.DecimalField(
+                decimal_places=4,
+                default=0,
+                max_digits=4,
+                validators=[
+                    django.core.validators.MinValueValidator(0),
+                    django.core.validators.MaxValueValidator(1),
+                ],
+            ),
+        ),
+        migrations.AlterField(
+            model_name="onchainpayment",
+            name="num_satoshis",
+            field=models.PositiveBigIntegerField(
+                null=True,
+                validators=[
+                    django.core.validators.MinValueValidator(0),
+                    django.core.validators.MaxValueValidator(7500000.0),
+                ],
+            ),
+        ),
+        migrations.AlterField(
+            model_name="onchainpayment",
+            name="sent_satoshis",
+            field=models.PositiveBigIntegerField(
+                null=True,
+                validators=[
+                    django.core.validators.MinValueValidator(0),
+                    django.core.validators.MaxValueValidator(7500000.0),
+                ],
+            ),
+        ),
+        migrations.AlterField(
+            model_name="onchainpayment",
+            name="swap_fee_rate",
+            field=models.DecimalField(decimal_places=2, default=1, max_digits=4),
+        ),
+    ]

api/models/ln_payment.py

@@ -1,4 +1,4 @@
-from decouple import config
+from django.conf import settings
 from django.contrib.auth.models import User
 from django.core.validators import MaxValueValidator, MinValueValidator
 from django.db import models
@@ -78,7 +78,7 @@ class FailureReason(models.IntegerChoices):
     num_satoshis = models.PositiveBigIntegerField(
         validators=[
             MinValueValidator(100),
-            MaxValueValidator(1.5 * config("MAX_TRADE", cast=int, default=1_000_000)),
+            MaxValueValidator(1.5 * settings.MAX_TRADE),
         ]
     )
     # Routing budget in PPM

api/models/market_tick.py

@@ -51,7 +51,7 @@ class MarketTick(models.Model):
     fee = models.DecimalField(
         max_digits=4,
         decimal_places=4,
-        default=config("FEE", cast=float, default=0),
+        default=0,
         validators=[MinValueValidator(0), MaxValueValidator(1)],
     )
 
@@ -71,7 +71,11 @@ def log_a_tick(order):
             premium = 100 * (price / market_exchange_rate - 1)
 
             market_tick = MarketTick.objects.create(
-                price=price, volume=volume, premium=premium, currency=order.currency
+                price=price,
+                volume=volume,
+                premium=premium,
+                currency=order.currency,
+                fee=config("FEE", cast=float, default=0),
             )
 
             return market_tick

api/models/onchain_payment.py

@@ -1,4 +1,4 @@
-from decouple import config
+from django.conf import settings
 from django.contrib.auth.models import User
 from django.core.validators import MaxValueValidator, MinValueValidator
 from django.db import models
@@ -7,9 +7,6 @@
 
 from control.models import BalanceLog
 
-MAX_TRADE = config("MAX_TRADE", cast=int, default=1_000_000)
-MIN_SWAP_AMOUNT = config("MIN_SWAP_AMOUNT", cast=int, default=1_000_000)
-
 
 class OnchainPayment(models.Model):
     class Concepts(models.IntegerChoices):
@@ -48,17 +45,11 @@ def get_balance():
 
     num_satoshis = models.PositiveBigIntegerField(
         null=True,
-        validators=[
-            MinValueValidator(0.5 * MIN_SWAP_AMOUNT),
-            MaxValueValidator(1.5 * MAX_TRADE),
-        ],
+        validators=[MinValueValidator(0), MaxValueValidator(1.5 * settings.MAX_TRADE)],
     )
     sent_satoshis = models.PositiveBigIntegerField(
         null=True,
-        validators=[
-            MinValueValidator(0.5 * MIN_SWAP_AMOUNT),
-            MaxValueValidator(1.5 * MAX_TRADE),
-        ],
+        validators=[MinValueValidator(0), MaxValueValidator(1.5 * settings.MAX_TRADE)],
     )
     # fee in sats/vbyte with mSats decimals fee_msat
     suggested_mining_fee_rate = models.DecimalField(
@@ -91,7 +82,7 @@ def get_balance():
     swap_fee_rate = models.DecimalField(
         max_digits=4,
         decimal_places=2,
-        default=config("MIN_SWAP_FEE", cast=float, default=0.01) * 100,
+        default=1,
         null=False,
         blank=False,
     )