<p><b>@tomhughes</b> requested changes on this pull request.</p>

<hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/2320#discussion_r304335519">db/migrate/20120219161649_add_user_image_fingerprint.rb</a>:</p>
<pre style='color:#555'>> @@ -1,4 +1,7 @@
 class AddUserImageFingerprint < ActiveRecord::Migration[4.2]
+  class User < ActiveRecord::Base
+  end
+
</pre>
<p>I don't think this will work if there are any users in the database because the migration relies on being able to access the images attached to the user via the paperclip attachment, which has not been declared here.</p>
<p>That of course gets even more fun once we remove paperclip from the Gemfile...</p>

<hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/2320#discussion_r304335785">db/migrate/20150111192335_subscribe_old_changesets.rb</a>:</p>
<pre style='color:#555'>> @@ -1,4 +1,7 @@
 class SubscribeOldChangesets < ActiveRecord::Migration[4.2]
+  class Changeset < ActiveRecord::Base
+  end
+
</pre>
<p>I think this will fail if there are changesets in the database as the migration relies on the <code>subscribers</code> and <code>user</code> associations on the changeset.</p>

<hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/2320#discussion_r304335967">db/migrate/20161011010929_subscribe_authors_to_diary_entries.rb</a>:</p>
<pre style='color:#555'>> @@ -1,4 +1,7 @@
 class SubscribeAuthorsToDiaryEntries < ActiveRecord::Migration[4.2]
+  class DiaryEntry < ActiveRecord::Base
+  end
+
</pre>
<p>Again the migration relies on the <code>subscriptions</code> and <code>user</code> associations.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/pull/2320?email_source=notifications&email_token=AAK2OLLAEUZYMFJ6ZX4G3S3P73Z67A5CNFSM4IEO2V72YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB6WDPEA#pullrequestreview-262944656">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLMCSTHVZGGDIUAS4D3P73Z67ANCNFSM4IEO2V7Q">mute the thread</a>.<img src="https://github.com/notifications/beacon/AAK2OLOYNEGH26BY5E7JJRDP73Z67A5CNFSM4IEO2V72YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB6WDPEA.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/pull/2320?email_source=notifications\u0026email_token=AAK2OLLAEUZYMFJ6ZX4G3S3P73Z67A5CNFSM4IEO2V72YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB6WDPEA#pullrequestreview-262944656",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/2320?email_source=notifications\u0026email_token=AAK2OLLAEUZYMFJ6ZX4G3S3P73Z67A5CNFSM4IEO2V72YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB6WDPEA#pullrequestreview-262944656",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>