Skip to content

Commit

Permalink
do not cache resources + save related fields for resources (#181)
Browse files Browse the repository at this point in the history
  • Loading branch information
Afani97 authored Mar 9, 2023
1 parent faa74f7 commit e46ba15
Show file tree
Hide file tree
Showing 2 changed files with 20 additions and 6 deletions.
9 changes: 8 additions & 1 deletion nc/admin.py
Original file line number Diff line number Diff line change
Expand Up @@ -45,14 +45,21 @@ def agency_name(self, obj):


class ResourceAdmin(admin.ModelAdmin):
list_display = ("title", "created_date",)
list_display = (
"title",
"created_date",
)

def save_model(self, request, obj, form, change):
super().save_model(request, obj, form, change)
if not obj.image:
obj.image = "forward-justice-logo"
obj.save()

def save_related(self, request, form, formsets, change):
super().save_model(request, form, formsets, change)
form.instance.agencies.set(form.cleaned_data["agencies"], clear=True)


admin.site.register(Agency, AgencyAdmin)
admin.site.register(StopSummary, StopSummaryAdmin)
Expand Down
17 changes: 12 additions & 5 deletions nc/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,8 @@
from django.core.mail import send_mail
from django.db.models import Case, Count, F, Q, Sum, Value, When
from django.db.models.functions import ExtractYear
from django.utils.decorators import method_decorator
from django.views.decorators.cache import never_cache
from django_filters.rest_framework import DjangoFilterBackend
from rest_framework import viewsets
from rest_framework.decorators import action
Expand Down Expand Up @@ -289,12 +291,17 @@ class ResourcesViewSet(viewsets.ReadOnlyModelViewSet):
def get_serializer_class(self):
return serializers.ResourcesSerializer(context={"request": self.request})

@method_decorator(never_cache)
def list(self, request, *args, **kwargs):
return Response({"results": self.serializer_class(
Resource.objects.all(),
many=True,
context={"request": self.request},
).data})
return Response(
{
"results": self.serializer_class(
Resource.objects.all(),
many=True,
context={"request": self.request},
).data
}
)


class ContactView(APIView):
Expand Down

0 comments on commit e46ba15

Please sign in to comment.