<div dir="ltr">Hi Dimitris,<div><br></div><div>I would definitely see this contribution as part of the core codebase rather that having it as contrib optional app. Therefore I'm +1 in keeping the js into layer_detail.html, while for css you could add it to the base.less (that then you can compile minified into base.css).</div><div><br></div><div>If you have tests, the better but being mostly js, as I suppose, it's ok without them as well as we don't have testing facilities in place for js.</div><div><br></div><div>While for the doc, it is in a dedicated folder into the main repo <a href="https://github.com/GeoNode/geonode/tree/master/docs">https://github.com/GeoNode/geonode/tree/master/docs</a>, you can find the most convenient place in there.</div><div><br></div><div>Thanks a lot for this, very nice and welcome feature.</div><div><br></div><div>Let us know if you need further info.</div><div><br></div><div>Ciao </div></div><div class="gmail_extra"><br><div class="gmail_quote">2017-09-19 10:33 GMT+02:00 Dimitris Karakostis <span dir="ltr"><<a href="mailto:karakostis.dimitris@gmail.com" target="_blank">karakostis.dimitris@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Hi all, </div><div><br></div><div>I have integrated the Layer Filtering functionality in my local master branch of GeoNode. The functionality gives the possibility to build basic queries based on the layer's attributes and values and download part of the layers (in the provided formats) (see screenshot). Currently it works for local, vector layers.</div><div><br></div><div>In the past we have discussed this functionality here: <span style="color:rgb(68,68,68);font-family:Roboto,Helvetica,Arial,sans-serif;font-size:13px"><a href="https://goo.gl/7rTDzF" target="_blank">https://goo.gl/7rTDzF</a></span></div><div><span style="color:rgb(68,68,68);font-family:Roboto,Helvetica,Arial,sans-serif;font-size:13px"><br></span></div><div><span style="color:rgb(68,68,68);font-family:Roboto,Helvetica,Arial,sans-serif;font-size:13px">As this is my first contribution to the GeoNode community, I want to be sure that the commit complies to best practices in terms of documentation, potential tests and code. </span></div><div><span style="color:rgb(68,68,68);font-family:Roboto,Helvetica,Arial,sans-serif;font-size:13px"><br></span></div><div><font color="#444444" face="Roboto, Helvetica, Arial, sans-serif">Currently I have included CSS and javascript code in the layer_detail.html. Should I move these snippets of code in external files?</font></div><div><font color="#444444" face="Roboto, Helvetica, Arial, sans-serif"><br></font></div><div><font color="#444444" face="Roboto, Helvetica, Arial, sans-serif">How and where do I update the documentation?</font></div><div><font color="#444444" face="Roboto, Helvetica, Arial, sans-serif"><br></font></div><div><font color="#444444" face="Roboto, Helvetica, Arial, sans-serif">Any advice on the above would be greatly appreciated.</font></div><div><font color="#444444" face="Roboto, Helvetica, Arial, sans-serif"><br></font></div><div><font color="#444444" face="Roboto, Helvetica, Arial, sans-serif">Thanks.</font></div><span class="HOEnZb"><font color="#888888"><div><font color="#444444" face="Roboto, Helvetica, Arial, sans-serif">Dimitris</font></div></font></span></div>
<br>______________________________<wbr>_________________<br>
geonode-devel mailing list<br>
<a href="mailto:geonode-devel@lists.osgeo.org">geonode-devel@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/geonode-devel" rel="noreferrer" target="_blank">https://lists.osgeo.org/<wbr>mailman/listinfo/geonode-devel</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Simone </div>
</div>