Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

FRED app rewrites leaflet config #49

Closed
nesnoj opened this issue Jul 4, 2019 · 6 comments · Fixed by #53
Closed

FRED app rewrites leaflet config #49

nesnoj opened this issue Jul 4, 2019 · 6 comments · Fixed by #53
Assignees
@Bachibouzouk @jh-RLI @christian-rli
Labels
bug Something isn't working

Comments

@nesnoj
Copy link
Member

nesnoj commented Jul 4, 2019 •
edited
Loading

Hey folks,

@christian-rli has put the FRED app on the WAMdev yesterday.
Unfortunately, your map coordinates (where the map is positioned on startup) defined in param DEFAULT_CENTER of LEAFLET_CONFIG override mine :( . (if I deactivate FRED, everything's fine)

This is a general WAM settings problem, see rl-institut/WAM#74

Hence, it's the same with other leaflet params as well such as TILES, SRID etc.

There seems to be a workaround for django-leaflet (different configs for multiple maps) but we need a general fix on WAM-level.
@nesnoj nesnoj added the bug Something isn't working label Jul 4, 2019
@nesnoj nesnoj mentioned this issue Jul 4, 2019
Open
@nesnoj nesnoj changed the title FRED app overrides leaflet config FRED app rewrites leaflet config Jul 4, 2019
@jh-RLI
Copy link
Collaborator

jh-RLI commented Jul 4, 2019

oh great I always wondered why my zoom level and initial coords are not changing, now I know its because I have the ego_vis and fred wam installed.

@christian-rli
Copy link
Collaborator

I have put FredWAM on the production server as well, btw... and indeed the same problems seem to occur. I'm going to take it off a gain until there is a fix for this.

@nesnoj
Copy link
Member Author

nesnoj commented Jul 4, 2019

Yes, please!!

@nesnoj
Copy link
Member Author

nesnoj commented Jul 4, 2019

Urgently

@christian-rli
Copy link
Collaborator

Sorry, forgot to rerun the docker after changing the YAML-file. It's back to normal now.

@nesnoj nesnoj mentioned this issue Jul 4, 2019
Merged
@nesnoj
Copy link
Member Author

nesnoj commented Jul 4, 2019 •
edited
Loading

Ok, implemented a solution for this specific django-leaflet-related issue in rl-institut/WAM_APP_stemp_abw#66 using a simple settings override.

You might wanna do the same..

@nesnoj nesnoj mentioned this issue Jul 5, 2019
Merged
@Bachibouzouk Bachibouzouk mentioned this issue Jul 8, 2019
Merged
@jh-RLI jh-RLI closed this as completed in #53 Jul 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Bachibouzouk Bachibouzouk

@jh-RLI jh-RLI

@christian-rli christian-rli

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix leaflet config interference rl-institut/WAM_APP_FRED
4 participants
@Bachibouzouk @nesnoj @jh-RLI @christian-rli