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

Labelling server queries page more clearly #1403

Merged
merged 2 commits into from Feb 23, 2016
Merged

Conversation

hflynn
Copy link
Member

@hflynn hflynn commented Feb 19, 2016

Addresses https://trac.openmicroscopy.org/ome/ticket/10907 by renaming the page and adding an overview box to signpost the content more clearly.

Open to suggestions for a better alternative link if anyone has one.

Will be staged at https://www.openmicroscopy.org/site/support/omero5.2-staging/developers/Server/Queries.html once built.

.. topic:: Overview

This page is aimed at internal server developers and does not
contain information of how to perform API queries. If that is the kind of
Copy link

Choose a reason for hiding this comment

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

for how ?

@hflynn hflynn added the develop label Feb 22, 2016
@joshmoore
Copy link
Member

I'd say this is definitely an improvement. This might actually be better suited in the javadoc itself, but rather than just making it disappear now, I say 👍

@hflynn
Copy link
Member Author

hflynn commented Feb 22, 2016

Did you mean the content better placed in the javadoc @joshmoore ? I.e. migrate the content and delete this page in future?

@joshmoore
Copy link
Member

@hflynn: exactly.

hflynn added a commit that referenced this pull request Feb 23, 2016
Labelling server queries page more clearly
@hflynn hflynn merged commit 415c14a into ome:develop Feb 23, 2016
@hflynn hflynn deleted the server-queries branch February 23, 2016 09:35
@sbesson sbesson added this to the 5.2.2 milestone Nov 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants