fix(project): update details

Author: tyronejosee

apps/blogs/admin.py

@@ -7,11 +7,6 @@
 from .choices import StatusChoices
 
 
-class TagInline(admin.TabularInline):
-    model = Post.tags.through
-    extra = 1
-
-
 @admin.register(Tag)
 class TagAdmin(BaseAdmin):
     """Admin for Tag model."""
@@ -34,7 +29,6 @@ class PostAdmin(BaseAdmin):
     list_filter = ["tags", "is_available"]
     readonly_fields = ["pk", "slug", "created_at", "updated_at"]
     ordering = ["title"]
-    inlines = [TagInline]
 
 
 @admin.register(PostReport)

apps/blogs/managers.py

@@ -8,7 +8,7 @@
 
 
 class TagManager(BaseManager):
-    """Manager for Post Model."""
+    """Manager for Tag Model."""
 
 
 class PostManager(BaseManager):
@@ -33,4 +33,4 @@ def get_search(self, search_term):
 
 
 class PostReportManager(BaseManager):
-    """Manager for Post Model."""
+    """Manager for PostReport Model."""

apps/blogs/views.py

@@ -26,6 +26,7 @@
 class TagListView(APIView):
     """
     View to list and create tags.
+    NOTE: This view has been refactored into an action of the viewsets.
 
     Endpoints:
     - GET api/v1/posts/tags/
@@ -57,6 +58,7 @@ def post(self, request):
 class PostListView(APIView):
     """
     View to list and create posts.
+    NOTE: This view has been refactored into an viewset.
 
     Endpoints:
     - GET api/v1/posts/
@@ -101,6 +103,7 @@ def post(self, request):
 class PostDetailView(APIView):
     """
     View to retrieve, update, and delete a post.
+    NOTE: This view has been refactored into an viewset.
 
     Endpoints:
     - GET api/v1/posts/{id}/
@@ -148,6 +151,7 @@ def delete(self, request, post_id):
 class PostSearchView(APIView):
     """
     View to search posts.
+    NOTE: This view has been refactored into an viewset.
 
     Endpoints:
     - GET api/v1/posts/?q={query}
@@ -179,7 +183,7 @@ def get(self, request):
 class FeaturedPostsView(APIView):
     """
     View to list featured posts.
-    # ! NOTE: This view has been refactored into an action of the viewsets.
+    NOTE: This view has been refactored into an action of the viewsets.
 
     Endpoints:
     - GET api/v1/posts/featured/
@@ -209,7 +213,7 @@ def get(self, request):
 class RecentPostsView(APIView):
     """
     View to list recent posts.
-    # ! NOTE: This view has been refactored into an action of the viewsets.
+    NOTE: This view has been refactored into an action of the viewsets.
 
     Endpoints:
     - GET api/v1/posts/recent/
@@ -239,7 +243,7 @@ def get(self, request):
 class PostReportView(APIView):
     """
     View for reporting a post.
-    # ! NOTE: This view has been refactored into an action of the viewsets.
+    NOTE: This view has been refactored into an action of the viewsets.
 
     Endpoints:
     - POST api/v1/posts/{id}/report/
@@ -286,7 +290,7 @@ def post(self, request, post_id):
 
 class ReportsView(APIView):
     """View for listing all reports.
-    # ! NOTE: This view has been refactored into an action of the viewsets.
+    NOTE: This view has been refactored into an action of the viewsets.
 
     Endpoints:
     - GET api/v1/posts/reports/

apps/blogs/viewsets.py

@@ -81,6 +81,7 @@ def create_report(self, request, *args, **kwargs):
                     status=status.HTTP_400_BAD_REQUEST,
                 )
 
+            # ! TODO: Move logic to the service layer
             with transaction.atomic():
                 # Update points of the post
                 post.points = post.points - 1

apps/drivers/signals.py

@@ -30,6 +30,6 @@ def delete_sensitive_documents(sender, instance, **kwargs):
                     )
                 )
         except FileNotFoundError:
-            # logger.error(f"File not found error")
             # ! TODO: Add sentry for logs
+            # logger.error(f"File not found error")
             pass

apps/jobs/models.py

@@ -11,7 +11,7 @@
 
 
 class Position(BaseModel):
-    """Model definition for JobPosition (Catalog)."""
+    """Model definition for Position."""
 
     position = models.CharField(max_length=100, unique=True)
     description = models.TextField()
@@ -26,7 +26,7 @@ def __str__(self):
 
 
 class Worker(BaseModel):
-    """Model definition for Worker (Entity)."""
+    """Model definition for Worker."""
 
     user_id = models.OneToOneField(User, on_delete=models.DO_NOTHING)
     phone_number = models.CharField(max_length=15)
@@ -59,7 +59,7 @@ def __str__(self):
 
 
 class Applicant(BaseModel):
-    """Model definition for Applicant (Entity)."""
+    """Model definition for Applicant."""
 
     user_id = models.OneToOneField(User, on_delete=models.CASCADE)
     phone_number = models.CharField(max_length=15)

roadmap.md

@@ -1,10 +1,5 @@
 # Roadmap
 
-## Home
-
-- [ ] `GET /home/kwords` Obtener todas las palabras más buscadas (AllowAny). [10]
-- [ ] `GET /home/restaurants` Obtener todas los restaurants más popolares (AllowAny). [10]
-
 ## Orders [OK]
 
 - [ ] `GET /orders/` Obtener todas las ordenes (IsAdministrator)
@@ -105,10 +100,8 @@ Pending implementation
 - [x] `DELETE /posts/{id}` Eliminar un post específico por su ID (IsMarketing).
 
 - [x] `POST /posts/{id}/report` Reportar un post específico por su ID (IsClient).
-- [ ] `GET /posts/tags` Obtener una lista de todas las etiquetas (IsMarketing).
-- [ ] `POST /posts/tags` Crear una nueva etiqueta (IsMarketing).
-
-TODO: recents and featured endpoint unificarlos con /posts agregando un filtro
+- [x] `GET /posts/tags` Obtener una lista de todas las etiquetas (IsMarketing).
+- [x] `POST /posts/tags` Crear una nueva etiqueta (IsMarketing).
 
 ## Jobs [OK]
 
@@ -117,12 +110,14 @@ TODO: recents and featured endpoint unificarlos con /posts agregando un filtro
 - [ ] `GET /positions/{id}` Obtener los detalles de una posición de trabajo específica por su ID (IsRRHH).
 - [ ] `PATCH /positions/{id}` Actualizar una posición de trabajo específica por su ID (IsRRHH).
 - [ ] `DELETE /positions/{id}` Eliminar una posición de trabajo específica por su ID (IsRRHH).
+
 - [ ] `GET /workers` Obtener una lista de todos los trabajadores (IsRRHH).
 - [ ] `POST /workers` Crear un nuevo trabajador (IsRRHH).
 - [ ] `GET /workers/{id}` Obtener los detalles de un trabajador específico por su ID (IsRRHH).
 - [ ] `PATCH /workers/{id}` Actualizar un trabajador específico por su ID (IsRRHH).
 - [ ] `DELETE /workers/{id}` Eliminar un trabajador específico por su ID (IsRRHH).
 - [ ] `POST /workers/{id}/terminate` Dar por terminado un contrato de trabajo (IsRRHH).
+
 - [ ] `POST /applicants/{id}/status` Actualizar el estado de una solicitud de trabajo específica (IsRRHH).
 - [ ] `GET /contracts/` Obtener una lista de todos los contratos de trabajo (IsRRHH).
 - [ ] `GET /contracts/{id}` Obtener los detalles de un contrato específico por su ID (IsRRHH).