make certain fields optional for listing

Author: jamesdoh0109

backend/market/management/commands/generate_listings.py

@@ -180,19 +180,6 @@ def handle(self, *args, **options):
             "House",
         ]
 
-        neighborhoods = [
-            "University City",
-            "Center City",
-            "Rittenhouse Square",
-            "Fairmount",
-            "Graduate Hospital",
-            "Old City",
-            "Northern Liberties",
-            "Fishtown",
-            "Queen Village",
-            "South Philadelphia",
-        ]
-
         street_names = [
             "Walnut",
             "Chestnut",
@@ -234,8 +221,8 @@ def handle(self, *args, **options):
             # generate random address
             street_number = random.randint(100, 4999)
             street = random.choice(street_names)
-            neighborhood = random.choice(neighborhoods)
-            address = f"{street_number} {street} St, {neighborhood}, Philadelphia, PA"
+
+            street_address = f"{street_number} {street} St, Philadelphia, PA"
 
             # random beds and baths
             beds = random.randint(0, 4)  # 0 for studio
@@ -271,7 +258,7 @@ def handle(self, *args, **options):
                     price=price,
                     negotiable=negotiable,
                     expires_at=expires_at,
-                    address=address,
+                    street_address=street_address,
                     beds=beds,
                     baths=baths,
                     start_date=start_date,

backend/market/migrations/0004_rename_address_sublet_street_address_and_more.py

@@ -0,0 +1,23 @@
+# Generated by Django 5.0.2 on 2026-02-02 20:37
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("market", "0003_delete_phoneverification"),
+    ]
+
+    operations = [
+        migrations.RenameField(
+            model_name="sublet",
+            old_name="address",
+            new_name="street_address",
+        ),
+        migrations.AlterField(
+            model_name="listing",
+            name="expires_at",
+            field=models.DateTimeField(blank=True, null=True),
+        ),
+    ]

backend/market/models.py

@@ -89,7 +89,7 @@ class Meta:
     )
     negotiable = models.BooleanField(default=True)
     created_at = models.DateTimeField(auto_now_add=True)
-    expires_at = models.DateTimeField()
+    expires_at = models.DateTimeField(null=True, blank=True)
 
     def __str__(self):
         return f"{self.title} by {self.seller}"
@@ -125,7 +125,7 @@ class Condition(models.TextChoices):
 
 
 class Sublet(Listing):
-    address = models.CharField(max_length=255)
+    street_address = models.CharField(max_length=255)
     beds = models.PositiveIntegerField()
     baths = models.PositiveIntegerField()
     start_date = models.DateField()

backend/market/serializers.py

@@ -2,10 +2,13 @@
 from django.core.exceptions import ValidationError as ModelValidationError
 from profanity_check import predict
 from rest_framework.serializers import (
+    BooleanField,
+    DateTimeField,
     ImageField,
     ModelSerializer,
     SerializerMethodField,
     SlugRelatedField,
+    URLField,
     ValidationError,
 )
 
@@ -97,7 +100,7 @@ def get_condition(self, obj):
 class SubletDataSerializer(ModelSerializer):
     class Meta:
         model = Sublet
-        fields = ["address", "beds", "baths", "start_date", "end_date"]
+        fields = ["street_address", "beds", "baths", "start_date", "end_date"]
 
 
 # Unified serializer for all listing types (Items and Sublets); used for CRUD operations
@@ -108,16 +111,31 @@ class ListingSerializer(ListingTypeMixin, ModelSerializer):
             "model": Item,
         },
         "sublet": {
-            "required_fields": ["address", "beds", "baths", "start_date", "end_date"],
+            "required_fields": [
+                "street_address",
+                "beds",
+                "baths",
+                "start_date",
+                "end_date",
+            ],
             "model": Sublet,
         },
     }
 
     images = ListingImageSerializer(many=True, required=False, read_only=True)
-    tags = SlugRelatedField(many=True, slug_field="name", queryset=Tag.objects.all())
+    tags = SlugRelatedField(
+        many=True,
+        slug_field="name",
+        queryset=Tag.objects.all(),
+        required=False,
+        allow_empty=True,
+    )
     seller = UserSerializer(read_only=True)
     listing_type = SerializerMethodField()
     additional_data = SerializerMethodField()
+    external_link = URLField(required=False, allow_blank=True, allow_null=True)
+    negotiable = BooleanField(required=False, default=True)
+    expires_at = DateTimeField(required=False, allow_null=True)
 
     class Meta:
         model = Listing
@@ -240,7 +258,7 @@ def _create_sublet(self, validated_data, additional_data):
         tags = validated_data.pop("tags", None)
 
         sublet = Sublet.objects.create(
-            address=additional_data.get("address"),
+            street_address=additional_data.get("street_address"),
             beds=additional_data.get("beds"),
             baths=additional_data.get("baths"),
             start_date=additional_data.get("start_date"),
@@ -297,7 +315,7 @@ def _update_item(self, instance, additional_data):
 
     def _update_sublet(self, instance, additional_data):
         sublet = instance.sublet
-        sublet_fields = ["address", "beds", "baths", "start_date", "end_date"]
+        sublet_fields = ["street_address", "beds", "baths", "start_date", "end_date"]
         for field in sublet_fields:
             if field in additional_data:
                 setattr(sublet, field, additional_data[field])

backend/market/views.py

@@ -1,6 +1,7 @@
 from django.conf import settings
 from django.contrib.auth import get_user_model
 from django.core.cache import cache
+from django.db.models import Q
 from django.utils import timezone
 from rest_framework import exceptions, mixins, status, viewsets
 from rest_framework.decorators import api_view, permission_classes
@@ -128,7 +129,7 @@ def get_filter_dict(listing_type):
         sublet_filters = {
             "beds": "sublet__beds",
             "baths": "sublet__baths",
-            "address": "sublet__address__icontains",
+            "address": "sublet__street_address__icontains",
         }
 
         if listing_type == "item":
@@ -168,7 +169,11 @@ def list(self, request, *args, **kwargs):
         if request.query_params.get("seller", "false").lower() == "true":
             queryset = queryset.filter(seller=request.user)
         else:
-            queryset = queryset.filter(expires_at__gte=timezone.now())
+            # Show listings that are not expired, or have no expiration
+            now = timezone.now()
+            queryset = queryset.filter(
+                Q(expires_at__gte=now) | Q(expires_at__isnull=True)
+            )
 
         page = self.paginate_queryset(queryset)
         if page is not None: