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.

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

Some issues with annotations #484

Closed
giohappy opened this issue Aug 30, 2021 · 9 comments
Closed

Some issues with annotations #484

giohappy opened this issue Aug 30, 2021 · 9 comments
Assignees
Labels
bug Something isn't working Epic
Milestone

Comments

@giohappy
Copy link

giohappy commented Aug 30, 2021

Annotations have been roughly integrated with #334 but I noticed some problems:

  • id and visibility are shown instead of Title and Description
    image
  • GeoNode stores and shows "Annotations" as a map layer. I guess this is because MapStore creates the "Annotations" layer inside the map configuration, and GeoNode blindly processes it as a nomral layer. @allyoucanmap does this layers has some specific info that could drive its filtering inside the geonode adapter? I don't want to filter it on the base of a hardcoded "Annotations" title.image
@giohappy giohappy added the bug Something isn't working label Aug 30, 2021
@giohappy giohappy added this to the 3.3.0 milestone Aug 30, 2021
@allyoucanmap
Copy link
Collaborator

GeoNode stores and shows "Annotations" as a map layer. I guess this is because MapStore creates the "Annotations" layer inside the map configuration, and GeoNode blindly processes it as a nomral layer. @allyoucanmap does this layers has some specific info that could drive its filtering inside the geonode adapter? I don't want to filter it on the base of a hardcoded "Annotations" title.

@giohappy I think we could try to filter out all the layer of type vector. The type vector is used for GeoJSON format inside mapstore

@giohappy
Copy link
Author

@giohappy I think we could try to filter out all the layer of type vector. The type vector is used for GeoJSON format inside mapstore

Ok... this should be done inside the django adapter I guess.

@allyoucanmap
Copy link
Collaborator

id and visibility are shown instead of Title and Description

@giohappy this issue seems related to a missing epic and reducers that are included only when the Annotation plugin is in the configuration. The epic setup the annotation layer adding a specific viewer to display the annotation results. I tried to add the Annotations plugin in the map preview but now I'm getting this results:

image

As you can see the button for editing are there even if we don't need those in a preview context and in this first investigation I did not find how to disable.

It seems that a map with annotations layers needs the Annotations plugin configured to make the identify work correctly.
A solution could be to add a config option for the Annotation plugin to disable the editing button in MapStore (if there are not yet)

@giohappy
Copy link
Author

giohappy commented Sep 17, 2021

A solution could be to add a config option for the Annotation plugin to disable the editing button in MapStore (if there are not yet)

Let's do it. Please open an issue for MapStore. Let's see if someone else can work on this.
@allyoucanmap please also open an issue for the other things, and give some indications on how / what to filter from the map config.

@giohappy
Copy link
Author

@allyoucanmap if I understand correctly (from the issue you opened on MapStore) the proposal is to exclude the Annotation plugin when in view mode.So your previous suggestion

A solution could be to add a config option for the Annotation plugin to disable the editing button in MapStore (if there are not yet)

is not necessary in this case, right?

@allyoucanmap
Copy link
Collaborator

@giohappy the issue is that it's missing the initialization process to add the correct viewer to annotation. Right now the correct viewer is applied only if the Annotation plugin is included in the localConfig. MapStore should apply the correct viewer even if the Annotation plugin is not included

@giohappy
Copy link
Author

as agreed @allyoucanmap we'll do the following:

@simboss
Copy link

simboss commented Oct 5, 2021

@allyoucanmap can you let me know where we are with this?

@allyoucanmap
Copy link
Collaborator

@simboss I worked on MapStore and proposed this solution that was accepted but the work is still pending. geosolutions-it/MapStore2#7332 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Epic
Projects
None yet
Development

No branches or pull requests

3 participants