Esri / storymap-series

The Story Map Series lets you present a series of maps via tabs, numbered bullets, or a side accordion.
Apache License 2.0
63 stars 59 forks source link

The geocoder configuration is not respected if anonymous access has been disabled #19

Open ghost opened 8 years ago

ghost commented 8 years ago

Please make the geocoder configurable like they do in Web AppBuilder so we can use our own geocoding service. Thanks.

glazou commented 8 years ago

We introduced with last release an optional geocoder configuration in the app/config.js file that should be identical to what WebAppBuilder is offering. Note that if you have configured custom geocoder in your Organization settings, those will be used.

See https://github.com/Esri/map-series-storytelling-template-js/blob/master/src/app/config.js#L473

ghost commented 8 years ago

We are not hosting the code, so changing JS file is not an option. We do have configured our custom geocoder in our org account settings ( thecityofcalgary.maps.arcgis.com) but the story map is not picking it up, and still using ESRI geocoder. See the attached screenshot.

On Fri, Mar 11, 2016 at 11:09 AM, glazou notifications@github.com wrote:

We introduced with last release an optional geocoder configuration in the app/config.js file that should be identical to what WebAppBuilder is offering. Note that if you have configured custom geocoder in your Organization settings, those will be used.

See https://github.com/Esri/map-series-storytelling-template-js/blob/master/src/app/config.js#L473

— Reply to this email directly or view it on GitHub https://github.com/Esri/map-series-storytelling-template-js/issues/19#issuecomment-195481943 .

glazou commented 8 years ago

Ok sorry, I found the issue. It appear that if your ArcGIS Online Organization has disabledanonymous access, the geocoder configuration defined is not applied properly. This is the setting:

image

Until we fix this with our next release planned for June 13th, you can either enable that setting or use the downloadable version and configure it manually.

I changed the title of your issue to reflect the issue and will update you here once fixed.

ghost commented 8 years ago

Thanks for looking into this. We don't want to allow public access of the site for various reasons. Hopefully you guys can fix this soon. My recommendation is to add a simple interface to allow user to set custom geocoder inside story map (cloud hosted), rather than getting it from AGOL as many orgs tend to have more than one custom geocoder for various purpose. We might create a custom geocoder just for this story map to make user search more relative.

On Mon, Mar 14, 2016 at 10:19 AM, glazou notifications@github.com wrote:

Ok sorry, I found the issue. It appear that if your ArcGIS Online Organization has disabledanonymous access, the geocoder configuration defined is not applied properly. This is the setting:

[image: image] https://cloud.githubusercontent.com/assets/994078/13751051/8940f216-e9c5-11e5-82bc-e9745cbd516d.png

Until we fix this with our next release planned for June 13th, you can either enable that setting or use the downloadable version and configure it manually.

I changed the title of your issue to reflect the issue and will update you here once fixed.

— Reply to this email directly or view it on GitHub https://github.com/Esri/map-series-storytelling-template-js/issues/19#issuecomment-196391820 .

glazou commented 8 years ago

Thanks, I did not realize this requirements. I would like to avoid redeveloping a full UI to configure geocoder in the builder but I can see a more limited UI just to select which of the geocoder configured in the Organization should be available in the story with checkbox. How does that sound?

ghost commented 8 years ago

That sounds good too. Thanks.

On Mon, Mar 14, 2016 at 12:26 PM, glazou notifications@github.com wrote:

Thanks, I did not realize this requirements. I would like to avoid redeveloping a full UI to configure geocoder in the builder but I can see a more limited UI just to select which of the geocoder configured in the Organization should be available in the story with checkbox. How does that sound?

— Reply to this email directly or view it on GitHub https://github.com/Esri/map-series-storytelling-template-js/issues/19#issuecomment-196455468 .

ghost commented 7 years ago

Do you have any update on this ? Thanks.

oevans commented 7 years ago

@darkbluemsu -- stay tuned for our release notes and blogs after tonight's release.