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

Reviewed GeoServer Quickstart #543

Merged
merged 8 commits into from
Jan 27, 2020
Merged

Reviewed GeoServer Quickstart #543

merged 8 commits into from
Jan 27, 2020

Conversation

flicstar
Copy link
Collaborator

@flicstar flicstar commented Nov 27, 2019

Reviewed as part of the QuickStart project during Google Season of Docs 2019. The aim is consistency across all OSGeoLive QuickStarts.
I added numbered steps and changed heading style and wording. Also modified the What Next? section. @fgdrf Please action comments and suggestions you agree with, and ignore those you don't.

Partner trac ticket link: https://trac.osgeo.org/osgeolive/ticket/2210

See this page for reference: https://trac.osgeo.org/osgeolive/wiki/How%20to%20document%20the%20quickstart%20file


#. Select the |osgeolive-appmenupath-geoserver| in the menu.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Replacement for |osgeolive-appmenupath-geoserver| as :menuselection:Geospatial --> Web Services --> GeoServer --> Start GeoServer has been introduced while shell-scripts can change installation path so we do not have to update docs.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, I will revert.

Comment on lines 270 to 280
Clients for WMS layers
================================================================================

A large variety of clients exist to make use of the `WMS <http://www.opengeospatial.org/standards/wms>`__ layers you are serving
from |GS|. This is a list of just some of them

* :doc:`uDig <../overview/udig_overview>`

* :doc:`OpenLayers <../overview/openlayers_overview>`

* :doc:`MapBender <../overview/mapbender_overview>`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any reason why this section has been removed. From my perspective it cross-reference other packages distrubuted by OSGeo live and its worth for users to see how these might work togeher

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, I will re-instate. It seemed out of place but now that you've explained it, it makes sense.

@camerons camerons merged commit 761cfa9 into OSGeo:master Jan 27, 2020
@flicstar flicstar deleted the geoserver branch February 12, 2020 00:22
@cvvergara cvvergara added this to the Release 14.0.0 milestone Aug 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants