<p>Great to see more progress on this <a class="user-mention" data-hovercard-type="user" data-hovercard-url="/users/openbrian/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/openbrian">@openbrian</a> !</p>
<p>I saw some things that caught my eye in the last few updates, for example:</p>
<ul>
<li>hardcoding webdriver stuff in the travis config, vs e.g. <a href="https://github.com/titusfortner/webdrivers">https://github.com/titusfortner/webdrivers</a></li>
<li>even more cucumber tests being added, vs using our existing test suites</li>
<li>lots of CSS and hardcoded layout stuff, vs using bootstrap for responsive layouts</li>
<li>other stuff we've discussed previously, like hardcoded values in factories</li>
</ul>
<p>I want to try to nudge you towards making this PR something that we can discuss and merge, rather than it ever-expanding project that is seemingly not approaching a merge-able state! Basically I have worries that this is following the same path as the GSoC Moderation branch, where it took me 11 months of work on the PR after the original contributor had moved on, and I don't want the same thing to happen again here.</p>
<p>What do you think? What can I do to help?</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/2390?email_source=notifications&email_token=AAK2OLIWMNA44QLBTR3OLW3RAQ2GBA5CNFSM4JAHYW42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKO3MZI#issuecomment-580761189">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLNTKZIC2MDCMAFSRGLRAQ2GBANCNFSM4JAHYW4Q">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AAK2OLI4E6KEBLTUTMD2XVTRAQ2GBA5CNFSM4JAHYW42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKO3MZI.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/2390?email_source=notifications\u0026email_token=AAK2OLIWMNA44QLBTR3OLW3RAQ2GBA5CNFSM4JAHYW42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKO3MZI#issuecomment-580761189",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/2390?email_source=notifications\u0026email_token=AAK2OLIWMNA44QLBTR3OLW3RAQ2GBA5CNFSM4JAHYW42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKO3MZI#issuecomment-580761189",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>