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

Ability to add WMS to project as a reference layer #1476

Open
wonderchook opened this issue May 5, 2017 · 9 comments
Open

Ability to add WMS to project as a reference layer #1476

wonderchook opened this issue May 5, 2017 · 9 comments

Comments

@wonderchook
Copy link
Contributor

Ability to add WMS to project as a reference layer

@dpalomino
Copy link

Partner provided a couple of examples that are public and they are using for adding layers (all open and free):

From the Environment Ministry:
http://geoservidor.minam.gob.pe/ArcGIS/services/ANP/MapServer/WMSServer?

From the Transport Ministry:
http://mtcgeo2.mtc.gob.pe:8080/geoserver/MTC_gis/wms?service=WMS&

@dpalomino
Copy link

They are not using any kind of authentication, so this will not be required.

@dpalomino
Copy link

Summarising the conversation with @oliverroick (thanks!)

  • There are some little difficulties managing WMS services working with several layers, if we want to enable/disable individually on the map, due to some Leaflet limitations
  • Partners will typically require to enable and disable these individually
  • There are two possible solutions for this:
    • Asking in separate UI instances the different layers contained in a single WMS service (as they'd be different services). This solution will not be elegant but would be acceptable.
    • If implementation effort is affordable, we will try to to have have in place to proper solution (i.e. being able to individually select the layers from the WMS)

UI was already drafted in Project Designer wireframes (to be reviewed with @clash99).

@wonderchook
Copy link
Contributor Author

@dpalomino with the WMS with different layers. When the WMS is registered could we register each layer separately? Is that what you are suggesting above?

@oliverroick
Copy link
Member

@wonderchook — Something along those lines. I thought that there should be a way where we enter the WMS base URL, get the capabilities document, generate a list of available layers and have the user select which layers they want to use. Some WMS provide a lot of layers, many might not be relevant for our partners, so we thought there should be a way to select the layers the should be included.

There are some areas I haven't thought about yet. For example, how we can represent that in the database, or how we can bake the different approaches for TMS and WMS into the same UI.

@dpalomino dpalomino changed the title Ability to add WMS to project as a reference layer (develop requirements) Ability to add WMS to project as a reference layer May 30, 2017
@dpalomino
Copy link

Using this issue to track the actual implementation, updating subject and scheduling for Sprint 18.

@clash99
Copy link
Contributor

clash99 commented Jun 26, 2017

Wireframes

PDF Document of Wireframes

Project Details - Map > Layers:

project map - layers

Project Details - Map > Layers - Add Layer:

project map - layers_ add modal

Project Details - Map > Layers - Select:

project map - layers_ select layers

@wonderchook
Copy link
Contributor Author

@clash99 I'm thinking that Basemaps and Layers should be together they are really a similar thing. Though I'm not sure how that changes the UI further down the line.

@clash99
Copy link
Contributor

clash99 commented Jun 28, 2017

@oliverroick - I made the changes to the above wireframes based on what we discussed earlier:

  • Moved layer selection to modal with concept of layer "group"
  • Removed tree view from index in favor of single level of layer groups

Let me know if you have any more changes or if I missed anything. Thanks!

@oliverroick oliverroick removed their assignment Sep 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants