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

4326 doc examples&replace fixes #4622

Merged
merged 13 commits into from
Apr 30, 2018
Merged

Conversation

mheppler
Copy link
Contributor

Related Issues

Pull Request Checklist

Unrelated to rest of branch - noticed a bug when building the docs. The
Windows page needed to be added to the index of the dev guide. Also
changed the page title to be more descriptive.
Added details about draft datasets and metadata to build appropriate
expectations and resolve user confusion.
Went through about half of our documentation to review use of examples.
Made various small tweaks. Second half forthcoming.
Finished my review of the way we use examples in docs. Lots of small
corrections.
Updated the example on the Install Prep page and fixed syntax on two
tables.
@coveralls
Copy link

coveralls commented Apr 26, 2018

Coverage Status

Coverage remained the same at 13.367% when pulling 07db977 on 4326-doc-examples&replace-fixes into c593125 on develop.


After successfully replacing a file, a new dataset draft version will be created. A summary of your actions will be recorded in the dataset Version Details table in the "Versions" tab on that dataset's page, and in the file Version Details table on the file's page. Both tables allow you to access all previous versions of the file across all previous versions of your dataset, including the old version of the file before you replaced it.
After successfully replacing a file, a new dataset draft version will be created. A summary of your actions will be recorded in the "Versions" tab on that dataset's page, and in "Versions" tab on the file's page. Both tabs allow you to access all previous versions of the file across all previous versions of your dataset, including the old version of the file before you replaced it.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggested revision...

A summary of your actions will be recorded in the "Versions" tab on both the dataset page and file page.

@@ -10,7 +10,7 @@ Finding Data
Without logging in to Dataverse, users can browse Dataverse, search for dataverses, datasets, and files, view dataset descriptions and files for
published datasets, and subset, analyze, and visualize data for published (restricted & not restricted) data files. To view an unpublished dataverse, dataset, or file, a user will need to be given permission from that dataverse's administrator to access it.

A user can search the dataverses, datasets, and files within a particular dataverse by using the search bar found on a dataverse page. For example, if you are on the Murray Research Archive Dataverse page, you can search that specific dataverse's contents by using the search bar and/or facets displayed on the page.
A user can search within a specific dataverse for the dataverses, datasets, and files it contains by using the search bar and facets found on that dataverse's page.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

"...found on that dataverse's page" looked odd to me. Suggested revision...

A user can use the search bar and facets to discover the dataverses, datasets, and files contained within a specific dataverse.

Copy link
Contributor Author

@mheppler mheppler left a comment

Choose a reason for hiding this comment

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

Added my $0.02 revision suggestions in comments on the commits view. Take them or leave. Approved for QA with or without further revision.

@@ -339,7 +339,7 @@ A more detailed "add" example using curl::

Example python code to add a file. This may be run by changing these parameters in the sample code:

* ``dataverse_server`` - e.g. https://dataverse.harvard.edu
* ``dataverse_server`` - e.g. https://dataverse.example.edu
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Would it be too much effort to use example.dataverse.org as an example, and then setup a redirect for that example domain to demo.dataverse.org? @pdurbin...??

Copy link
Member

Choose a reason for hiding this comment

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

@mheppler good idea. Fixed in 83b8683

A couple of small rephrasings from Code Review suggestions.
@pdurbin
Copy link
Member

pdurbin commented Apr 26, 2018

Please no ampersands in branch names. I can work around this easily but check this out:

$ git checkout 4326-doc-examples&replace-fixes
[1] 98185
-bash: replace-fixes: command not found
murphy:guides.dataverse.org pdurbin$ error: pathspec '4326-doc-examples' did not match any file(s) known to git.

[1]+  Exit 1                  git checkout 4326-doc-examples

Thanks.

Copy link
Member

@pdurbin pdurbin left a comment

Choose a reason for hiding this comment

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

"Foobar College" is weird but otherwise this looks fine.

@@ -695,7 +695,7 @@ See :ref:`Branding Your Installation` above.

By default the footer says "Copyright © [YYYY]" but you can add text after the year, as in the example below.

``curl -X PUT -d ", The President & Fellows of Harvard College" http://localhost:8080/api/admin/settings/:FooterCopyright``
``curl -X PUT -d ", Foobar College" http://localhost:8080/api/admin/settings/:FooterCopyright``
Copy link
Member

Choose a reason for hiding this comment

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

Foobar College?

@@ -10,7 +10,7 @@ Finding Data
Without logging in to Dataverse, users can browse Dataverse, search for dataverses, datasets, and files, view dataset descriptions and files for
published datasets, and subset, analyze, and visualize data for published (restricted & not restricted) data files. To view an unpublished dataverse, dataset, or file, a user will need to be given permission from that dataverse's administrator to access it.

A user can search the dataverses, datasets, and files within a particular dataverse by using the search bar found on a dataverse page. For example, if you are on the Murray Research Archive Dataverse page, you can search that specific dataverse's contents by using the search bar and/or facets displayed on the page.
A user can search within a specific dataverse for the dataverses, datasets, and files it contains by using the search bar and facets displayed on that dataverse's page.
Copy link
Member

Choose a reason for hiding this comment

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

I'm glad this MRA example was taken out. Nothing against the archive, it's just most people haven't heard of it.

@kcondon kcondon merged commit 07db977 into develop Apr 30, 2018
@poikilotherm poikilotherm deleted the 4326-doc-examples&replace-fixes branch July 1, 2020 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants