From b5e3ada2221f97e96e4c91e7f47a7c32392ca145 Mon Sep 17 00:00:00 2001
From: Kieran Eglin <kieran.eglin@gmail.com>
Date: Sat, 24 Feb 2024 12:30:47 -0800
Subject: [PATCH] Updated docs

---
 lib/pinchflat/media.ex | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/lib/pinchflat/media.ex b/lib/pinchflat/media.ex
index f2547de..ccd6e48 100644
--- a/lib/pinchflat/media.ex
+++ b/lib/pinchflat/media.ex
@@ -150,6 +150,11 @@ defmodule Pinchflat.Media do
   @doc """
   Deletes the media_item's associated files. Will leave the media_item in the database.
 
+  NOTE: this deletes the metadata files as well, but maybe it shouldn't? I'm wondering if
+  the metadata is more a concern of the DB record itself and should be lumped in with those
+  delete operations. But the metadata does come from the download operation of the file.
+  Food for thought but not a priority at the moment.
+
   Returns {:ok, %MediaItem{}}
   """
   def delete_attachments(media_item) do