Geonode: DEBUG_STATIC=True breaks frontend

Created on 2 Oct 2017  路  6Comments  路  Source: GeoNode/geonode

Setting DEBUG_STATIC=True will break frontend in many places (all dynamic lists, like layers, docs, maps etc).

This is because with that setting, angular.js is included in scripts, but it's not present in static files. There's angular.min.js, but I guess it defeats the purpose of DEBUG_STATIC:

https://github.com/GeoNode/geonode/blob/master/geonode/templates/index.html#L192

Most helpful comment

Ok, let's add a warning when DEBUG_STATIC is enabled then. It's not obvious that enabling this flag requires npm.

All 6 comments

@cezio did you also run bower install? It is required for js development

Good point. No, i haven't. But in this case, what's the point of having minified version of angular anyway?

The goal is all the js dependencies are shipped in the assets.js file and that we don't ship any of them in the geonode's static folder. When needed to develop on static, turning on DEBUG_STATIC and running bower install should set up all that's needed.

Ok, let's add a warning when DEBUG_STATIC is enabled then. It's not obvious that enabling this flag requires npm.

@cezio I think this can be closed, right? Would you?

@francbartoli can you close this ticket?

Was this page helpful?
0 / 5 - 0 ratings

Related issues

HaidarZ picture HaidarZ  路  4Comments

audetrobergem picture audetrobergem  路  9Comments

dpadron picture dpadron  路  7Comments

t-book picture t-book  路  9Comments

gannebamm picture gannebamm  路  4Comments