From bec46e15a97f7533c2ee1562a82e7c9f3603296e Mon Sep 17 00:00:00 2001
From: Jeremy Daer <jeremy@37signals.com>
Date: Tue, 9 Dec 2025 00:36:39 -0800
Subject: [PATCH] Drop defunct Account upload attachments (#2030)

Controller and routes removed in 6a62df470cfcdacbe193716344a3b26bbb766fba
---
 app/models/account.rb                         |   2 -
 app/views/uploads/create.json.jbuilder        |   4 -
 .../convert_markdowns_to_rich_text.rb         | 115 ------------------
 3 files changed, 121 deletions(-)
 delete mode 100644 app/views/uploads/create.json.jbuilder
 delete mode 100755 script/migrations/convert_markdowns_to_rich_text.rb

diff --git a/app/models/account.rb b/app/models/account.rb
index 61a636adb..0bb4d4111 100644
--- a/app/models/account.rb
+++ b/app/models/account.rb
@@ -10,8 +10,6 @@ class Account < ApplicationRecord
   has_many :columns, dependent: :destroy
   has_many :exports, class_name: "Account::Export", dependent: :destroy
 
-  has_many_attached :uploads
-
   before_create :assign_external_account_id
   after_create :create_join_code
 
diff --git a/app/views/uploads/create.json.jbuilder b/app/views/uploads/create.json.jbuilder
deleted file mode 100644
index 984afba49..000000000
--- a/app/views/uploads/create.json.jbuilder
+++ /dev/null
@@ -1,4 +0,0 @@
-json.message "File uploaded successfully"
-json.fileName @upload.filename.to_s
-json.mimetype @upload.content_type
-json.fileUrl @upload.slug_url
diff --git a/script/migrations/convert_markdowns_to_rich_text.rb b/script/migrations/convert_markdowns_to_rich_text.rb
deleted file mode 100755
index e093730f6..000000000
--- a/script/migrations/convert_markdowns_to_rich_text.rb
+++ /dev/null
@@ -1,115 +0,0 @@
-#!/usr/bin/env ruby
-
-require_relative "../config/environment"
-require "redcarpet"
-require "nokogiri"
-
-class ActionText::Markdown < ApplicationRecord
-  belongs_to :record, polymorphic: true
-end
-
-class MarkdownToActionTextConverter
-  ATTACHMENT_URL_REGEX = %r{/u/(?<slug>[^\/\s\)]+)}
-
-  def initialize(html)
-    @doc = Nokogiri::HTML::DocumentFragment.parse(html)
-    @attachments = []
-  end
-
-  def convert
-    process_images
-    process_links
-    [ @doc.to_html, @attachments ]
-  end
-
-  private
-    def process_images
-      @doc.css("img").each do |img|
-        src = img["src"].presence
-        if src && match = src.match(ATTACHMENT_URL_REGEX)
-          if (attachment = find_attachment(match[:slug]))
-            img.replace(build_attachment_node(attachment))
-            @attachments << attachment
-          end
-        end
-      end
-    end
-
-    def process_links
-      @doc.css("a").each do |link|
-        href = link["href"].presence
-
-        if href && match = href.match(ATTACHMENT_URL_REGEX)
-          if (attachment = find_attachment(match[:slug]))
-            link.replace(build_attachment_node(attachment))
-            @attachments << attachment
-          end
-        end
-      end
-    end
-
-    def build_attachment_node(attachment)
-      html = ActionText::Attachment.from_attachable(attachment).to_html
-      fragment = Nokogiri::HTML::DocumentFragment.parse(html)
-
-      node = fragment.at_css("action-text-attachment")
-      node["url"] = Rails.application.routes.url_helpers.rails_blob_path(attachment.blob, only_path: true)
-
-      fragment
-    end
-
-    def find_attachment(slug)
-      ActiveStorage::Attachment.find_by(slug: slug)
-    end
-end
-
-class RedcarpetRenderer
-  def self.render(markdown)
-    renderer = Redcarpet::Render::HTML.new
-    markdowner = Redcarpet::Markdown.new(renderer,
-      autolink: true,
-      tables: true,
-      fenced_code_blocks: true,
-      strikethrough: true,
-      superscript: true,
-    )
-    markdowner.render(markdown.to_s)
-  end
-end
-
-def process_all(klass, field)
-  klass.find_each do |record|
-    markdown = ActionText::Markdown.find_by(record: record, name: field)
-    next unless markdown
-
-    puts "markdown.id=#{markdown.id}"
-    next unless markdown.record
-
-    html = RedcarpetRenderer.render(markdown.content.to_s)
-    converter = MarkdownToActionTextConverter.new(html)
-    rich_text_html, attachments = converter.convert
-
-    rich_text = ActionText::RichText.create!(
-      name: markdown.name,
-      record: markdown.record,
-      body: rich_text_html
-    )
-
-    attachments.each do |attachment|
-      attachment.update!(record: rich_text)
-    end
-
-    puts "✓ Created rich text for #{markdown.record_type}##{markdown.record_id} (#{markdown.name})"
-  rescue => e
-    warn "✗ Failed to process markdown ##{markdown.id}: #{e.class} - #{e.message}"
-  end
-end
-
-ApplicationRecord.with_each_tenant do |tenant|
-  puts "Processing tenant: #{tenant}"
-
-  ActionText::RichText.delete_all
-
-  process_all(Card, :description)
-  process_all(Comment, :body)
-end