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

User Guide: Permissions + Publish Dataset/Dataverse Need More Info #2653

Closed
2 tasks
eaquigley opened this issue Oct 19, 2015 · 16 comments
Closed
2 tasks

User Guide: Permissions + Publish Dataset/Dataverse Need More Info #2653

eaquigley opened this issue Oct 19, 2015 · 16 comments

Comments

@eaquigley
Copy link
Contributor

Need to update the user guide to reflect changes based on features, functionality, and suggestions from users.

Update List:

  • Permissions: write up more information about each specific permission
  • Publish Dataset + Dataverse: add more information about the software will never automatically publish for you, end user needs to do that on their own
@eaquigley eaquigley self-assigned this Oct 19, 2015
@eaquigley eaquigley added this to the 4.3 milestone Oct 19, 2015
@pdurbin
Copy link
Member

pdurbin commented Oct 19, 2015

@eaquigley just a heads up that soon I'll be creating a new 4.2.1 branch from the 4.3 branch and I might make it the default.

@pdurbin
Copy link
Member

pdurbin commented Oct 20, 2015

@eaquigley I just created the 4.2.1 branch so if you plan to put this issue through QA for the 4.2.1 milestone, please put your commits in the 4.2.1 branch and change the milestone to 4.2.1.

@pdurbin
Copy link
Member

pdurbin commented Oct 26, 2015

@eaquigley I don't know if you plan to cover in the guides the fact that roles can be created but @kcondon just reported how it's possible to create roles with combinations of permissions that lead to poor usability in my opinion, such as having a role with only the "ViewUnpublishedDataset" permission: #2648 (comment)

@bencomp
Copy link
Contributor

bencomp commented Oct 29, 2015

As Dataverse ships with a default set of roles, may I suggest to explain these roles first? Normal users, including admins, will not be able to create (or update/delete) roles. Relevant roles should be explained at the various levels of role assignment, i.e. both in "Dataverse management" and "Dataset + File management".

Before making major changes in the user guide, however, the role assignment pages should be given appropriate thought as requested/described in #2255, #2354 (likely dupe of #2255), #2241 (likely dupe of #2232) and possibly other issues. For example, for DataverseNL partners it is more obvious to grant users the Dataset Creator role than the Contributor role, but that is not a "short-hand option" like Curator and Contributor are.

@pdurbin
Copy link
Member

pdurbin commented Oct 30, 2015

Related: #2616

@mercecrosas mercecrosas modified the milestones: 4.3, In Review Nov 30, 2015
@scolapasta scolapasta modified the milestone: Not Assigned to a Release Jan 28, 2016
@mheppler mheppler changed the title User Guide: Update Based on Feature Changes + Suggestions User Guide: Permissions + Publish Dataset/Dataverse Need More Info Oct 6, 2016
@pdurbin
Copy link
Member

pdurbin commented Jun 29, 2017

Two things. People continue to struggle with permissions, such as yesterday at https://groups.google.com/d/msg/dataverse-community/OebwRJZwDHY/4LxOf_BSBAAJ .

Also, I'm consolidating the following ticket into this one and I think it's a good idea we should consider: Permissions: List roles in order of power of permissions #3040

@pdurbin pdurbin added the User Role: Curator Curates and reviews datasets, manages permissions label Jul 4, 2017
dlmurphy added a commit that referenced this issue Sep 19, 2017
Completely rewrote the Dataverse Management page's sections on
permissions. Note: This commit still includes a set of outdated
screenshots that have already been replaced on a different branch that
has yet to reach production.

Also, fixed an error that was appearing on the config page.
@dlmurphy
Copy link
Contributor

dlmurphy commented Sep 21, 2017

EDIT: This comment has been made into a new issue, #4148

I've encountered a usability issue while writing docs for dataset permissions. Paging @TaniaSchlatter and @mheppler for their consideration.

The file-level permissions page can only be accessed through the dataset page, like so:
filepermissions1

--
This link brings you to this page:
filepermissions2

On this page, you can click "Grant Access to Users/Groups" to see this modal:
filepermissions3

Here you can grant specific users access to specific restricted files. As far as I can tell, this is the only place you can do this.

I expected to be able to do this via the file page for each specific file, but I can't find any option to do that:
filepermissions4

I recommend we include a menu option where I've drawn that arrow, called "Permissions", that takes the user to a page where they can grant specific users access to this file.

Now that we have file pages, it's completely unintuitive to force users to go to the dataset level to grant access to a specific file.

@scolapasta
Copy link
Contributor

@dlmurphy , could you open that as a separate issue?

It is true, and technically we could use some aspect of the current ManagePermissionsPage as it could be used for a file. It may not be ideal, but would be the easiest.

dlmurphy added a commit that referenced this issue Sep 21, 2017
Rewrote docs for dataset and file permissions, and made further edits to
Dataverse Permissions docs.
@dlmurphy
Copy link
Contributor

@scolapasta Good call, I created issue #4148 for this. I'll put that info from your comment in there too.

dlmurphy added a commit that referenced this issue Sep 21, 2017
Various misc. typo fixes.
dlmurphy added a commit that referenced this issue Sep 21, 2017
Fixed conflict at src/main/java/Bundle.properties - both my edits and
the edits from develop live in harmony
@dlmurphy dlmurphy removed their assignment Sep 21, 2017
@dlmurphy dlmurphy removed the User Role: Curator Curates and reviews datasets, manages permissions label Sep 21, 2017
@scolapasta scolapasta assigned dlmurphy and unassigned scolapasta Sep 22, 2017
dlmurphy added a commit that referenced this issue Sep 22, 2017
Made some edits and additions to the permissions docs based on
@scolapasta 's review.
@dlmurphy dlmurphy removed their assignment Sep 22, 2017
dlmurphy added a commit that referenced this issue Sep 22, 2017
A couple further minor edits based on conversation with @scolapasta
dlmurphy added a commit that referenced this issue Sep 22, 2017
Fixed a couple syntax errors in Sphinx.
@kcondon kcondon self-assigned this Sep 25, 2017
pdurbin added a commit that referenced this issue Sep 25, 2017
Conflicts (trailing content in both cases):
doc/sphinx-guides/source/installation/config.rst
src/main/java/Bundle.properties
kcondon added a commit that referenced this issue Sep 25, 2017
Permissions Messaging and Documentation [#2653]
@kcondon kcondon closed this as completed Sep 25, 2017
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

10 participants