planetfederal / django-osgeo-importer

Do not use this repo, use https://github.com/GeoNode/django-osgeo-importer
GNU General Public License v3.0
2 stars 1 forks source link

instead of committing minified deps, automate and document build process as part of install #4

Closed ghost closed 8 years ago

ghost commented 8 years ago

I think it's a mess to include angular wizard in minified form in the repo.

referring to the build of static assets that make up the JS front end.

sarasafavi commented 8 years ago

Agree 100%. This would likely also take care of https://github.com/boundlessgeo/django-osgeo-importer/issues/3

ghost commented 8 years ago

package.json seems pretty useless, I guess they were downloading minified angular and just checking it into the repo?

sarasafavi commented 8 years ago

Fixed in #7

ghost commented 8 years ago

Decided not to require any frontend steps for an install, it seems counterproductive for a django app and I would prefer to avoid forcing the usage of any particular asset management strategy since django projects that might want this will use whatever they want to.