Esri / crowdsource-reporter

An ArcGIS Online group application template authored by organization and made available to constituents to report a problem or observation.
http://solutions.arcgis.com/local-government/help/crowdsource-reporter/
Apache License 2.0
25 stars 32 forks source link

Add method for sharing app #157

Closed allisonmuise closed 5 years ago

allisonmuise commented 8 years ago

specifically - a specific view into the app - the current layer, report, etc

allisonmuise commented 6 years ago

https://community.esri.com/ideas/15104-photo-app-for-survey123-collector

allisonmuise commented 6 years ago

Add a configurable option for sharing the current view of the app. This share icon should appear to the right of the Sign in text, and to the left of the help button. On mobile devices it should be in the burger menu.

see #354 for URL parameter details.

ashishchoure commented 6 years ago

@allisonmuise

Here is our approach to implement the share functionality.

On click of share icon, share dialog will be opened with text box for showing the application URL. The URL will contain parameters such as "webmap", "layerid" and "oid" based on following conditions :

Here is the screenshot of share dialog: image

The URL shown in the screenshot in just a sample. We are planning to show the bitly URL in the text box.

Please let us know in case of any addition or changes.

allisonmuise commented 6 years ago

this looks good @ashishchoure . I will get back to you on using bitly and the share icon after talking to the AGOL team about what they use.

allisonmuise commented 6 years ago

@ashishchoure I am sending you an email conversation that contains links to preferred alternates to bitly and the share graphic.

ashishchoure commented 6 years ago

@allisonmuise

As per the email conversation, here is the screenshot for the share icon in application header (highlighted in orange color). image

Let us know your thoughts.

allisonmuise commented 6 years ago

@ashishchoure looks good!

allisonmuise commented 5 years ago

verified

skrishn commented 5 years ago

@allisonmuise : added testcase