@grekko commented on this pull request.


> @@ -39,6 +49,10 @@ class Note < ApplicationRecord
 
   DEFAULT_FRESHLY_CLOSED_LIMIT = 7.days
 
+  def comments_with_extra_open_comment

@gravitystorm I've updated the code for the Data-Migration to immediately 
destroy the first comment as part of the migration and wrapped the whole 
migration into a transaction.

I have trouble understanding what you are proposing in regards to the logic in 
the API which needs to prepend the first comment for notes. Could you have a 
look at my [previous 
comment](https://github.com/openstreetmap/openstreetmap-website/pull/4481#discussion_r1790139682)?

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4481#discussion_r1790376242
You are receiving this because you are subscribed to this thread.

Message ID: 
<openstreetmap/openstreetmap-website/pull/4481/review/2352186...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to