For #111 : dynamic layers in Wegue format#113
For #111 : dynamic layers in Wegue format#113justb4 wants to merge 5 commits intowegue-oss:masterfrom
Conversation
chrismayer
left a comment
There was a problem hiding this comment.
Thanks for this @justb4. Two questions:
1.) Why was it necessary to delete test/unit/specs/ol/Map.spec.js
2.) Any reason you removed the alias me for this in the existing code? Creates a lot of unrelated diffs in your changeset.
|
@chrismayer sorry, your questions escaped me. ad 1) ad 2) Ok strictly not part of issue, but cleanup which I considered too minor for issue/PR. |
|
Thanks for clarification. One further point: Due to asynchronous handling the position in the |
|
Hmm, yes. The Layers coming from the Solution would be to "stack" any remaining regular Layers (not adding them) after the As an alternative, for progress, we could merge this PR and open a new issue for " |
Introduces a new Wegue Layer type
LAYERCOLLECTIONwhich is mainly a URL pointing to a remote endpoint (or file local to the Wegue App) that contains an array of Wegue Layers (which in theory could even again containLAYERCOLLECTIONs).fetch()) is asynchronous, several functions are nowasync/await(ES2015 syntax).Later similar "expanding layer types" could be added for WMS, WFS etc (via GetCapabilities) and even OGC OpenAPI Features ("WFSv3").