<p><a class="user-mention" data-hovercard-type="user" data-hovercard-url="/hovercards?user_id=41215187" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/RoyallDesigns">@RoyallDesigns</a> Thanks for your PR. If we decide to make the redaction title required, then there are three things that should be changed:</p>
<ul>
<li>The database needs to have a "not null" constraint added on the <code>title</code> column. This will require a <a href="https://edgeguides.rubyonrails.org/active_record_migrations.html" rel="nofollow">migration</a> to be added. The migration might also need to check for existing records with blank titles, and set a default title, before adding the constraint.</li>
<li>The model needs to have a validation, which is what you've got here</li>
<li>The model test (in test/models/redaction_test.rb) should have a new test, asserting that the redaction is not valid when the title is empty. This ensures that any refactoring mistakes that happen to the model in future get caught in the test suite.</li>
</ul>

<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/2034#issuecomment-432534816">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABWnLWFkWs8P8E96HsuQm-ZT66ojw5e6ks5uoA5NgaJpZM4X3FF3">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABWnLdgFNFUn7DQnpN_fpj1hKvx5cvaJks5uoA5NgaJpZM4X3FF3.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/openstreetmap/openstreetmap-website","title":"openstreetmap/openstreetmap-website","subtitle":"GitHub repository","main_image_url":"https://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/openstreetmap/openstreetmap-website"}},"updates":{"snippets":[{"icon":"PERSON","message":"@gravitystorm in #2034: @RoyallDesigns Thanks for your PR. If we decide to make the redaction title required, then there are three things that should be changed:\r\n\r\n* The database needs to have a \"not null\" constraint added on the `title` column. This will require a [migration](https://edgeguides.rubyonrails.org/active_record_migrations.html) to be added. The migration might also need to check for existing records with blank titles, and set a default title, before adding the constraint.\r\n* The model needs to have a validation, which is what you've got here\r\n* The model test (in test/models/redaction_test.rb) should have a new test, asserting that the redaction is not valid when the title is empty. This ensures that any refactoring mistakes that happen to the model in future get caught in the test suite."}],"action":{"name":"View Pull Request","url":"https://github.com/openstreetmap/openstreetmap-website/pull/2034#issuecomment-432534816"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/pull/2034#issuecomment-432534816",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/2034#issuecomment-432534816",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
},
{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB",
"title": "Re: [openstreetmap/openstreetmap-website] Added a validation to the Redaction model for the title. (#2034)",
"sections": [
{
"text": "",
"activityTitle": "**Andy Allan**",
"activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png",
"activitySubtitle": "@gravitystorm",
"facts": [

]
}
],
"potentialAction": [
{
"name": "Add a comment",
"@type": "ActionCard",
"inputs": [
{
"isMultiLine": true,
"@type": "TextInput",
"id": "IssueComment",
"isRequired": false
}
],
"actions": [
{
"name": "Comment",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"IssueComment\",\n\"repositoryFullName\": \"openstreetmap/openstreetmap-website\",\n\"issueId\": 2034,\n\"IssueComment\": \"{{IssueComment.value}}\"\n}"
}
]
},
{
"name": "Close pull request",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"PullRequestClose\",\n\"repositoryFullName\": \"openstreetmap/openstreetmap-website\",\n\"pullRequestId\": 2034\n}"
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/openstreetmap/openstreetmap-website/pull/2034#issuecomment-432534816"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"MuteNotification\",\n\"threadId\": 400314743\n}"
}
],
"themeColor": "26292E"
}
]</script>