<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small"><pre style="white-space:pre-wrap;color:rgb(0,0,0)">Dear Alessio</pre><pre style="white-space:pre-wrap;color:rgb(0,0,0)"><br></pre><pre style="white-space:pre-wrap;color:rgb(0,0,0)">Thank you for your message below. I really do agree with everything you laid out below and I will get Rizky to retract the PR in favor of more atomic ones. Some things are going to be a bit tricky to keep atomic as there are a lot of inter-dependencies but we will do our best to split things up nicely so it is clear what each improvement does.</pre><pre style="white-space:pre-wrap;color:rgb(0,0,0)">I would really appreciate your help / like to take you up on your offer to review the PR's when we split them up as it is going to be a big headache to keep them all current to master if we have 10 or more PR's sitting in the queue for a long time.</pre><pre style="white-space:pre-wrap;color:rgb(0,0,0)">So thank you for your inputs and stand by for a fresh batch of PR's.</pre><pre style="white-space:pre-wrap;color:rgb(0,0,0)"><br></pre><pre style="white-space:pre-wrap;color:rgb(0,0,0)">Regards</pre><pre style="white-space:pre-wrap;color:rgb(0,0,0)"><br></pre><pre style="white-space:pre-wrap;color:rgb(0,0,0)">Tim</pre><pre style="white-space:pre-wrap;color:rgb(0,0,0)"><br></pre><pre style="white-space:pre-wrap;color:rgb(0,0,0)">> Dear all,
> I would like to know if someone of the devs is taking care about this PR.

> At the current state it cannot be merged of course. Other than this I see
> people keeping pushing commits on that branch and therefore updating the PR
> which since a long time is completely misaligned from master branch.

> Other issues preventing this PR to be merged are:

> - more than 150 files modified; it is almost impossible to do a review of
such huge amount of files

> - no connection to issues; each PR should solve a single problem and must
be clear how to reproduce the issue and/or how to test the code. No one can
spend months testing the whole GeoNode functionalities from scratch.

> I know that the GeoNode community has no explicit rules yet on the
management or PRs, but usually in order to be merged a PR you should:

> - close this huge one.

> - split it into several smaller PRs related to specific issues opened on
GeoNode

> - whenever is possible attach test cases or at least explain how to test
the code

> - manage to keep each PR aligned with the code and Travis to pass the build

> I'll be happy to help you merge the PRs as soon as possible after reviewing
them.

> Regards,

> Alessio Fabiani</pre></div><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div style="text-align:center">------------------------------------------------------------------------------------------</div><div style="text-align:center"><div style="color:rgb(0,0,0);font-family:arial,helvetica,sans-serif;font-size:small;display:inline">​</div><img src="https://mail.google.com/mail/u/1/?ui=2&ik=7ebbf93252&view=fimg&th=15a9e43c0bbf93ca&attid=0.1&disp=emb&realattid=9436e64174762963_0.1.1&attbid=ANGjdJ-WL_J24nvBiYidJ0V0xpdsYqRwswi5nq5oywcJEHC7twYVMrxG0y-GNPu_ahJthV8ktXwYuHoVm5DI2FrRAtPzelTRHrsE2wweR9uu8wT-zz2hNRqFwi4SorQ&sz=s0-l75-ft&ats=1488721675176&rm=15a9e43c0bbf93ca&zw&atsh=1" style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px;"><br></div><div style="text-align:center">Tim Sutton</div><div style="text-align:center"><span style="text-align:start">Visit </span><a href="http://kartoza.com/" style="text-align:start" target="_blank">http://kartoza.com</a><span style="text-align:start"> to find out about open source:</span><br style="text-align:start"><span style="text-align:start"> * Desktop GIS programming services</span><br style="text-align:start"><span style="text-align:start"> * Geospatial web development</span></div><div style="text-align:center"><span style="text-align:start">* GIS Training</span></div><div style="text-align:center"><span style="text-align:start">* Consulting Services</span></div><div style="text-align:center"><div style="text-align:center">Skype: timlinux Irc: timlinux on #qgis at <a href="http://freenode.net/" target="_blank">freenode.net</a></div><div style="text-align:start"><div style="text-align:center">Tim is a member of the QGIS Project Steering Committee</div><div style="text-align:center">-------------------------------------------------------------------------------------------</div><div style="text-align:center">Kartoza is a merger between Linfiniti and Afrispatial</div></div></div></div></div></div></div>
</div>