Merge pull request #3216 from ResearchHub/feed-cleanup

Feed app cleanup

Author: gzurowski

src/feed/admin.py

@@ -36,13 +36,6 @@ def has_comments(metrics: dict) -> bool:
     return replies > 0 or review_count > 0
 
 
-def get_content_type_name(feed_entry) -> str:
-    try:
-        return feed_entry.content_type.model.lower()
-    except (AttributeError, TypeError):
-        return "unknown"
-
-
 def parse_iso_datetime(date_string: str) -> Optional[datetime]:
     if not date_string or not isinstance(date_string, str):
         return None

src/feed/document_serializers/__init__.py

@@ -16,7 +16,6 @@
     get_age_hours_from_content,
     get_bounties_from_content,
     get_comment_count_from_metrics,
-    get_content_type_name,
     get_fundraise_amount_from_content,
     get_peer_review_count_from_metrics,
     get_social_media_engagement_from_metrics,
@@ -63,15 +62,6 @@ def test_has_comments(self):
         metrics = {"replies": 0, "review_metrics": {"count": 0}}
         self.assertFalse(has_comments(metrics))
 
-    def test_get_content_type_name(self):
-        """Test extracting content type name from feed entry."""
-        # Mock feed_entry with content_type
-        feed_entry = Mock()
-        feed_entry.content_type.model = "paper"
-
-        result = get_content_type_name(feed_entry)
-        self.assertEqual(result, "paper")
-
     def test_parse_iso_datetime(self):
         """Test parsing ISO 8601 datetime strings."""
         # Test with Z suffix

src/feed/document_serializers/feed_document_serializer.py

@@ -4,15 +4,13 @@
 
 from .activity_feed_view import ActivityFeedViewSet
 from .feed_view import FeedViewSet
-from .feed_view_v2__DEPRECATED import FeedV2ViewSet
 from .funding_feed_view import FundingFeedViewSet
 from .grant_feed_view import GrantFeedViewSet
 from .journal_feed_view import JournalFeedViewSet
 
 __all__ = [
     "ActivityFeedViewSet",
     "FeedViewSet",
-    "FeedV2ViewSet",
     "FundingFeedViewSet",
     "GrantFeedViewSet",
     "JournalFeedViewSet",