Marta Miranda / boletin_api

precio_admin y precio es exactamente lo mismo, por que haces esto?

almaceno en la variable el ultimo objeto creado por el admin para pasarlo a la funcion "calculadora" donde realiza un nuevo calculo y almacenar su valor en el elemento monto

aquí ve a comprar directamente: edicto.usuario != self.request.user

Added 1 commit:

8762df5c - correcciones de errores en perfom_create

Por favor, unificar la condición: obj.vigencia_desde <= today and not precios_activos.exists():

Aquí puedes directamente utilizar viewsets.ModelViewSet

Identico a al comentario anterior

Por favor utiliza otra forma de representar, cuerpo_edicto podría ser una "sin fin" de caracteres y no tiene demasiado sentido.

Added 1 commit:

4fd465cb - correcion de api, models, y admin

Status changed to closed

  class PagoViewSets(mixins.CreateModelMixin,
                     mixins.RetrieveModelMixin,
                     mixins.UpdateModelMixin,
                     mixins.DestroyModelMixin,
                     mixins.ListModelMixin,
                     viewsets.GenericViewSet):
      serializer_class = PagoSerializer
      permission_classes = [IsAuthenticated, IsAdminOrAuthorized]
      queryset = ComprobantePago.objects.all()
      def perform_create(self, serializer):
          try:
              edicto = Edicto.objects.get(estado='aprobado')
              precio_admi = Precio.objects.latest('id')

                     mixins.UpdateModelMixin,
                     mixins.DestroyModelMixin,
                     mixins.ListModelMixin,
                     viewsets.GenericViewSet):
      serializer_class = PagoSerializer
      permission_classes = [IsAuthenticated, IsAdminOrAuthorized]
      queryset = ComprobantePago.objects.all()
      def perform_create(self, serializer):
          try:
              edicto = Edicto.objects.get(estado='aprobado')
              precio_admi = Precio.objects.latest('id')
              precio = Precio.objects.get(id=precio_admi.id)
              monto = calculadora(edicto, precio)
              usuario_actual = self.request.user
              if edicto.usuario != usuario_actual:

  from django.contrib import admin
  from django.core.exceptions import ValidationError
  from django.utils import timezone
  from .models import Edicto, Precio
  # Register your models here.
  admin.site.register(Precio)
  class PrecioAdmin(admin.ModelAdmin):
      def save_model(self, request, obj, form, change):
          precios_activos = Precio.objects.filter(vigencia_hasta__gte=timezone.now().date())
          if not change:
              today = timezone.now().date()
              if obj.vigencia_desde <= today:

      ordering_fields = ('usuario', )
      ordering = 'usuario'
      queryset = Precio.objects.all()
  class PagoViewSets(mixins.CreateModelMixin,

          try:
              edicto = Edicto.objects.get(estado='pendiente_de_pago')
              precio = Precio.objects.latest('id')
              monto = calculadora(edicto, precio)
              if edicto.usuario != self.request.user:
                  raise UsuarioNoAutorizado
              serializer.validated_data['monto'] = monto
              serializer.validated_data['edicto'] = edicto
              serializer.save()
          except Edicto.DoesNotExist:
              raise EdictoNoEncontrado
          except Precio.DoesNotExist:
              raise PrecioNoEncontrado
  class ComprobanteViewSets(mixins.CreateModelMixin,

Marta Miranda / boletin_api

Feature/#48 crear dos enpoints para comprobantepago

Closed by Enzo Yair 2023-09-08 14:39:37 UTC

Feature/#48 crear dos enpoints para comprobantepago

Check out, review, and merge locally

Closed by Enzo Yair 2023-09-08 14:39:37 UTC //<![CDATA[ $('.js-timeago-pending').removeClass('js-timeago-pending').timeago() //]]>

Closed by Enzo Yair 2023-09-08 14:39:37 UTC