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

Add Data from URL #11

Closed
torrin47 opened this issue Sep 21, 2015 · 23 comments
Closed

Add Data from URL #11

torrin47 opened this issue Sep 21, 2015 · 23 comments
Assignees
Milestone

Comments

@torrin47
Copy link
Contributor

I believe Vincent's widget is most of the way there:
https://github.com/USEPA/AddLayerFromWeb-widget-js
Though in his readme, it seems like there might be issues with how it interacts with our TOC. Let's leave this issue open until the TOC matures some.

@torrin47 torrin47 added the tools label Sep 21, 2015
@David-Yarnell
Copy link
Contributor

When you say Add data from Url do you mean from a published services of some type (Dynamic Map Service Image Services etc.)? There is an add service widget (https://geonet.esri.com/docs/DOC-6263) that seem to work well.

@brosenbaum
Copy link
Contributor

Yes, that's what we mean - add service as rest or wms service
(raster,map,feature). kml is not as important.

Barb

On Tue, Sep 22, 2015 at 9:28 AM, David Yarnell notifications@github.com
wrote:

When you say Add data from Url do you mean from a published services of
some type (Dynamic Map Service Image Services etc.)? There is an add
service widget (https://geonet.esri.com/docs/DOC-6263) that seem to work
well.


Reply to this email directly or view it on GitHub
#11 (comment)
.

Barbara Rosenbaum
Innovate!, Inc.
Office: (541) 929-6622
Message: (703) 922.9090 x 739
brosenbaum@innovateteam.com

Innovate!, Inc. is a Virginia-based 8(a), Hispanic- and woman-owned
business delivering Geospatial, Information Technology, Security, Business
Consulting, Project Management and Executive Coaching solutions to our
government and private sector clients. Check us out at www.innovateteam.com.

@torrin47
Copy link
Contributor Author

The big bonus of Vincent's widget is being able to search for services in
ArcGISOnline, including non-public GeoPlatform items- maybe less of a
requirement for EA, but still kinda handy.
also a requirement for EA.

On Tue, Sep 22, 2015 at 2:58 PM, brosenbaum notifications@github.com
wrote:

Yes, that's what we mean - add service as rest or wms service
(raster,map,feature). kml is not as important.

Barb

On Tue, Sep 22, 2015 at 9:28 AM, David Yarnell notifications@github.com
wrote:

When you say Add data from Url do you mean from a published services of
some type (Dynamic Map Service Image Services etc.)? There is an add
service widget (https://geonet.esri.com/docs/DOC-6263) that seem to work
well.


Reply to this email directly or view it on GitHub
<
#11 (comment)

.

Barbara Rosenbaum
Innovate!, Inc.
Office: (541) 929-6622
Message: (703) 922.9090 x 739
brosenbaum@innovateteam.com

Innovate!, Inc. is a Virginia-based 8(a), Hispanic- and woman-owned
business delivering Geospatial, Information Technology, Security, Business
Consulting, Project Management and Executive Coaching solutions to our
government and private sector clients. Check us out at
www.innovateteam.com.


Reply to this email directly or view it on GitHub
#11 (comment)
.

Torrin Hultgren
Senior Manager, Geospatial Solutions
Innovate!, Inc.
O: (703) 922-9090 x737

http://www.innovateteam.com

Innovate!, Inc. is a Virginia-based 8(a), Hispanic- and woman-owned
business delivering Geospatial, Information Technology, Security,
Business Consulting, Project Management and Executive Coaching
solutions to our government and private sector clients. Check us out
at www.innovateteam.com.

@torrin47
Copy link
Contributor Author

So as we've learned, Vincent's widget does not do this, only the much older Esri Flex widget. We can possibly leverage that code. Barb has tracked it down.

@torrin47
Copy link
Contributor Author

For the purposes of this ticket, let's go with a simple add from URL functionality. Add from GeoPlatform will be a separate issue down the road.

@torrin47 torrin47 modified the milestones: Milestone 5, Milestone 6 Dec 23, 2015
@Baohong Baohong modified the milestone: Milestone 6 Jan 7, 2016
@torrin47 torrin47 modified the milestone: Milestone 7 Jan 13, 2016
David-Yarnell added a commit that referenced this issue Jan 13, 2016
Conflicts:
	widgets/Addfile/Widget.html
@David-Yarnell
Copy link
Contributor

Question, I have this widget added but one thing that I have found is that we have to add the domain of the service that you want to enable to be added to the widget config file. Here is a link that explains it (https://geonet.esri.com/docs/DOC-6263). I also looked at Vincent's widget and it look like it requires the same. If we want to use this widget is there a list of servers that we want to allow the user to add?

@torrin47
Copy link
Contributor Author

Huh, that's odd. I guess we can experiment with wildcards, otherwise we
might have to do our best to enable a reasonable short list of known
servers and add some text to the UI that says to let us know if they find
one we haven't enabled. A little clunky, but doesn't seem to be much
choice. I'm not seeing anything in the widget that generates a friendly
error if a user tries adding a URL not on the list - maybe we could add
that as a contribution to the general widget community?

On Thu, Jan 14, 2016 at 1:50 PM, David Yarnell notifications@github.com
wrote:

Question, I have this widget added but one thing that I have found is that
we have to add the domain of the service that you want to enable to be
added to the config file. Here is a link that explains it (
https://geonet.esri.com/docs/DOC-6263). I also looked at Vincent's widget
and it look like it requires the same. If we want to use this widget is
there a list of servers that we want to allow the user to add?


Reply to this email directly or view it on GitHub
#11 (comment)
.

Torrin Hultgren
Senior Manager, Geospatial Solutions
Innovate!, Inc.
O: (703) 922-9090 x737

http://www.innovateteam.com

Innovate!, Inc. is a Virginia-based 8(a), Hispanic- and woman-owned
business delivering Geospatial, Information Technology, Security,
Business Consulting, Project Management and Executive Coaching
solutions to our government and private sector clients. Check us out
at www.innovateteam.com.

@torrin47
Copy link
Contributor Author

Possibly add ability to email EnviroAtlas support requesting new hostname with 1-click approach.

@torrin47 torrin47 modified the milestones: Milestone 7, Milestone 8 Feb 4, 2016
torrin47 added a commit that referenced this issue Feb 16, 2016
# Conflicts:
#	config.json
#	widgets/AddService/config.json
#	widgets/Addfile/Widget.js
David-Yarnell referenced this issue Feb 17, 2016
Merge remote-tracking branch 'refs/remotes/origin/master' into Add-Da…
@brosenbaum
Copy link
Contributor

In regard to the URL issue for the user's added service, would it be an option to start "tracking/recording" URLs the user adds to the Flex App?? Even sending an email with the user input? Might mean some modification to the code, but could be a way to start getting a list together?

@Baohong
Copy link
Contributor

Baohong commented Feb 22, 2016

It might be possible to record the domains in a log.txt file through Flex code.

torrin47 added a commit that referenced this issue Mar 3, 2016
@torrin47
Copy link
Contributor Author

torrin47 commented Mar 3, 2016

Additional desired functionality - new ticket?

  • Allow users to enter email address for notification from our team when URL has been added
  • Do we need to warn about HTTPS?
  • Add text to clarify different service types (short term) or autodetect service types based on URL characteristics (long term)

@torrin47
Copy link
Contributor Author

torrin47 commented Mar 9, 2016

Before we close this ticket, let's

  • Add text to clarify different service types
  • Allow users to enter email address for notification from our team when URL has been added
  • Setup node mailer on leb server.

@David-Yarnell
Copy link
Contributor

How do you guys feel about the below for service types text:

Dynamic Map Service - Web service that generates maps
Tiled/Cashed Map Service - A cache of static images
Image Service - A raster dataset or mosaic dataset
WMS - A service compliant with the OGC Web Map Service (WMS) specification

@torrin47
Copy link
Contributor Author

Wow - so hard to be concise without resorting to corporate jargon! You've done well, but somehow I do think it's necessary to specify that the Map and Image service are Esri or ArcGIS. And could we hyperlink? I'm looking at this page in Esri's help and thinking we could probably borrow from them, including their links to more info:
http://doc.arcgis.com/en/arcgis-online/create-maps/add-layers.htm

Dynamic Map Service - Esri ArcGIS Server Web service that generates maps (http://doc.arcgis.com/en/arcgis-online/reference/arcgis-server-services.htm)
Tiled/Cashed Map Service - A Tile layer is a set of web-accessible image tiles that reside on a server. (http://doc.arcgis.com/en/arcgis-online/reference/tile-layers.htm)
Image Service - Esri ArcGIS Server Web service that provides access to imagery (http://doc.arcgis.com/en/arcgis-online/reference/arcgis-server-services.htm)
WMS - A service that follows the OGC Web Map Service specification. (http://doc.arcgis.com/en/arcgis-online/reference/ogc.htm)

Thoughts? Too much?

@David-Yarnell
Copy link
Contributor

That is a lot of content for that space. I like the idea of the links. I'm
going to stub that text in and see what it looks like.

On Wed, Mar 16, 2016 at 4:29 PM, Torrin Hultgren notifications@github.com
wrote:

Wow - so hard to be concise without resorting to corporate jargon! You've
done well, but somehow I do think it's necessary to specify that the Map
and Image service are Esri or ArcGIS. And could we hyperlink? I'm looking
at this page in Esri's help and thinking we could probably borrow from
them, including their links to more info:
http://doc.arcgis.com/en/arcgis-online/create-maps/add-layers.htm

Dynamic Map Service - Esri ArcGIS Server Web service that generates maps (
http://doc.arcgis.com/en/arcgis-online/reference/arcgis-server-services.htm
)
Tiled/Cashed Map Service - A Tile layer is a set of web-accessible image
tiles that reside on a server. (
http://doc.arcgis.com/en/arcgis-online/reference/tile-layers.htm)
Image Service - Esri ArcGIS Server Web service that provides access to
imagery (
http://doc.arcgis.com/en/arcgis-online/reference/arcgis-server-services.htm
)
WMS - A service that follows the OGC Web Map Service specification. (
http://doc.arcgis.com/en/arcgis-online/reference/ogc.htm)

Thoughts? Too much?


You are receiving this because you were assigned.
Reply to this email directly or view it on GitHub
#11 (comment)

David Yarnell
Geospatial Developer
Innovate! Inc.
Phone:(208)610-9981

@David-Yarnell
Copy link
Contributor

I have the text in with the links (the links are images) and I think it looks ok.
image

@torrin47
Copy link
Contributor Author

Yeah, that looks great to me, thanks so much!

On Thu, Mar 17, 2016 at 9:03 AM, David Yarnell notifications@github.com
wrote:

I have the text in with the links (the links are images) and I think it
looks ok.
[image: image]
https://cloud.githubusercontent.com/assets/5278901/13852338/e18ecce4-ec1e-11e5-8dd0-332d6cfdfb78.png


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#11 (comment)

Torrin Hultgren
Senior Manager, Geospatial Solutions
Innovate!, Inc.
O: (703) 922-9090 x737

http://www.innovateteam.com

Innovate!, Inc. is a Virginia-based 8(a), Hispanic- and woman-owned
business delivering Geospatial, Information Technology, Security,
Business Consulting, Project Management and Executive Coaching
solutions to our government and private sector clients. Check us out
at www.innovateteam.com.

@David-Yarnell
Copy link
Contributor

So one thing I may have missed is identifying when the user tries to add a service with http. I'm assuming we are going to need that?

@brosenbaum
Copy link
Contributor

yep, we will!

On Thu, Mar 17, 2016 at 9:53 AM, David Yarnell notifications@github.com
wrote:

So one thing I may have missed is identifying when the user tries to add a
service with http. I'm assuming we are going to need that?


You are receiving this because you commented.
Reply to this email directly or view it on GitHub
#11 (comment)

Barbara Rosenbaum
Innovate!, Inc.
Office: (541) 929-6622
Message: (703) 922.9090 x 739
brosenbaum@innovateteam.com

Innovate!, Inc. is a Virginia-based 8(a), Hispanic- and woman-owned
business delivering Geospatial, Information Technology, Security, Business
Consulting, Project Management and Executive Coaching solutions to our
government and private sector clients. Check us out at www.innovateteam.com.

@torrin47
Copy link
Contributor Author

Above input box perhaps there could be text that says:

Enter full URL of service:

On error:

It looks like this service is not CORS-enabled (https://en.wikipedia.org/wiki/Cross-origin_resource_sharing). Click below to send the URL for this service to the EnviroAtlas support team for further investigation. You may optionally add your email address to be notified when updates have been made.

@brosenbaum
Copy link
Contributor

I like the suggestion of a more thorough description - and that it shows up "on error" . here are my edit's to Torrin's text:

"The service URL provided is not CORS-enabled (https://en.wikipedia.org/wiki/Cross-origin_resource_sharing). Click below to send the URL for this service to the EnviroAtlas support team for further investigation. Including your email address is optional."

Also - if I can make a suggestion on the email box text? - change the wording above to say:
"For notification about the service inclusion status, please provide your email address."

just in case we need to reject something...??

And from yesterday's discussion of adding individual layers rather than the whole service... This was a new topic/request so do we make this a new issue?

Thanks!

@torrin47
Copy link
Contributor Author

Entered as issue #102

@brosenbaum
Copy link
Contributor

enviroatlas@epa.gov

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants