Skip to content
This repository has been archived by the owner on Jun 27, 2023. It is now read-only.

Can't view map template #191

Open
bssher opened this issue Aug 20, 2018 · 8 comments
Open

Can't view map template #191

bssher opened this issue Aug 20, 2018 · 8 comments
Assignees
Labels

Comments

@bssher
Copy link

bssher commented Aug 20, 2018

I did the following.

Add django-geonode-client to your requirements.txt Add geonode-client to your INSTALLED_APPS

For GeoNode
Change the LAYER_PREVIEW_LIBRARY to react

But I don't see the react map template in my new maps. What step am I missing?

@bssher
Copy link
Author

bssher commented Aug 21, 2018

Update, I got the template to load, but receive the following error. I am using Geonode 2.9

TemplateSyntaxError at /maps/new

'future' is not a registered tag library. Must be one of:

@afabiani
Copy link
Member

It is not a GeoNode issue but a geonode-client one.

It depends on this

https://github.com/GeoNode/geonode-client/blob/master/geonode-client/templates/geonode-client/map_new.html#L2

future is not allowed anymore on Django 1.11.11 as far as I know.

@afabiani afabiani reopened this Aug 22, 2018
@afabiani
Copy link
Member

sorry, closed by mistake. reopening

@bartvde
Copy link
Collaborator

bartvde commented Aug 22, 2018

@milafrerichs any ideas on this one?

@t-book
Copy link

t-book commented Aug 23, 2018

Since it seems to be removed with Django 1.9, GeoNode 2.8 should work without problems (@bssher can you confirm?) . For 2.9 we could check if we can just remove it and use url directly as the error suggests?

@bssher
Copy link
Author

bssher commented Aug 23, 2018 via email

@t-book
Copy link

t-book commented Aug 23, 2018

@bssher since Django 1.9 url is autoloaded. Just tested with master branch - as said you have to remove the future tag.

In 2.8.0 this problem does not occur as here Django 1.8 is in use. I do not want to make a PR right now as this would break 2.8 ?! @bartvde @milafrerichs maybe it would be good to setup a branch for each target version or some conditional import?

@milafrerichs
Copy link
Collaborator

hi, thanks for reporting this. I will take a look and maybe prepare a PR and tags for specific versions.

@milafrerichs milafrerichs self-assigned this Aug 24, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

5 participants