Great contribution Dimitri! +1 in the code base<br><br><signature>Sent from <a href="https://n1.nylas.com/link/72e0e9e7634d6c6f2a502dc7068f5361d905ee3269feda10a8072c3c9d4d1f86/0?redirect=https%3A%2F%2Fnylas.com%3Fref%3Dn1&recipient=geonode-devel%40lists.osgeo.org">Nylas Mail</a>, the best free email app for work</signature><img class="n1-open" width="0" height="0" style="border:0; width:0; height:0;" src="https://n1.nylas.com/open/72e0e9e7634d6c6f2a502dc7068f5361d905ee3269feda10a8072c3c9d4d1f86?recipient=geonode-devel%40lists.osgeo.org">
          <div class="gmail_quote nylas-quote nylas-quote-id-ed7134ae45fa3dc6750c3d42be6774758adec52be3916654a758ee13c7685e6b">
            <br>
            On set 19 2017, at 1:04 pm, Simone Dalmasso <simone.dalmasso@gmail.com> wrote:
            <br>
            <blockquote class="gmail_quote"
              style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
              <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?recipient=geonode-devel%40lists.osgeo.org">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><br /><div>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 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?recipient=geonode-devel%40lists.osgeo.org" 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>Currently I have included CSS and javascript code in the layer_detail.html. Should I move these snippets of code in external files?</div><div><br /></div><div>How and where do I update the documentation?</div><div><br /></div><div>Any advice on the above would be greatly appreciated.</div><div><br /></div><div>Thanks.</div><span><div>Dimitris</div></span></div>
<br />______________________________<wbr></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?recipient=geonode-devel%40lists.osgeo.org" rel="noreferrer" target="_blank">https://lists.osgeo.org/<wbr></wbr>mailman/listinfo/geonode-devel</a><br />
<br /></blockquote></div><br /><br /><div><br /></div>-- <br /><div>Simone </div>
</div>
<pre class="nylas-plaintext">_______________________________________________
geonode-devel mailing list
geonode-devel@lists.osgeo.org
https://lists.osgeo.org/mailman/listinfo/geonode-devel
</pre>
            </blockquote>
          </div>