Skip to content

Conversation

@robfrank
Copy link
Owner

@robfrank robfrank commented Dec 1, 2025

This pull request introduces several significant improvements to the content management features in the backend, focusing on robustness and feature completeness. Notably, it implements the ability to delete content, adds an endpoint to refresh content, and enhances the download logic with retry capabilities. The changes also include updates to controllers, service layers, permissions, and tests to support these new features.

Content Management Features

  • Implemented DeleteContentUseCase with a full stack (controller, service, command, interface) allowing deletion of content by link ID. [1] [2] [3] [4] Fdb06b47L139R139, Fdb06b47L150R150, [5]
  • Added a "refresh content" endpoint and business logic to re-trigger content download for a given link, including controller and service updates. [1] [2] [3] [4] Fdb06b47L105R105)

Robustness Improvements

  • Enhanced DownloadContentService with retry logic (3 retries, 1s delay) for content downloads to improve reliability. [1] [2]

Controller and API Updates

  • Updated GetContentController to support both standard content retrieval and refresh operations; added DeleteContentController. [1] Fdb06b47L139R139, Fdb06b47L150R150, [2]
  • Adjusted API route permission checks to differentiate between GET, POST (refresh), and DELETE operations for content endpoints.

Testing Enhancements

  • Added and updated tests for new controller methods (refreshContent and deletion), including mocks for new service dependencies. [1] [2] [3]

Documentation

  • Updated ROADMAP.md to mark the new features and improvements as completed, with brief implementation notes.

@mergify
Copy link
Contributor

mergify bot commented Dec 1, 2025

🧪 CI Insights

Here's what we observed from your CI run for 1ef412c.

🟢 All jobs passed!

But CI Insights is watching 👀

@codacy-production
Copy link

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
-0.68% (target: -1.00%) 29.49%
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (f2025da) 2348 1188 50.60%
Head commit (1ef412c) 2418 (+70) 1207 (+19) 49.92% (-0.68%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#458) 78 23 29.49%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

@robfrank robfrank merged commit c4b88f1 into main Dec 1, 2025
9 of 10 checks passed
@robfrank robfrank deleted the feat/content-domain branch December 1, 2025 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants