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

Make the error message for users who may not add datasets/-verses more explicit #6708

Closed
BPeuch opened this issue Feb 28, 2020 · 12 comments
Closed
Labels
Feature: Accessibility Accessibility is the practice of making your websites usable by as many people as possible. Feature: Permissions Type: Suggestion an idea UX & UI: Design This issue needs input on the design of the UI and from the product owner

Comments

@BPeuch
Copy link
Contributor

BPeuch commented Feb 28, 2020

Version: 4.19

Hello everybody,

When a user is not allowed to add datasets or dataverses, clicking on the Add Data button in the upper toolbar leads to a 403 "Not Authorized" page which I feel could be a bit more explicitly worded:

not_authorized

I think it would be more user-friendly to have something along the lines of: "You do not have permission to add new data[sets/verses]. You can use the Contact button to request permission from the [name of Dataverse] administrator."

@youssefOuahalou explained to me that this is a generic error message and that having a more specific message would require some fine tuning.

However I think it would make things more explicit

@pdurbin
Copy link
Member

pdurbin commented Feb 28, 2020

@BPeuch I totally agree. Can you please post a screenshot of what the error looks like when you use the other (older) "Add Data" button? This one:

Screen Shot 2020-02-28 at 11 26 21 AM

@BPeuch
Copy link
Contributor Author

BPeuch commented Feb 28, 2020

Well that's the thing, @pdurbin: this other button simply does not appear when you don't have authorization, although the one in the toolbar does. =p

no_add_data

I guess the rationale behind hiding this button was: "If they don't have the authorization, then we shouldn't let them believe that they may do it and then confront them with an error message." And then maybe the newer button, in the toolbar, was overlooked.

But either way, I side with your point of view: let users see that it's about adding data, and then encourage them to sign up.

@pdurbin
Copy link
Member

pdurbin commented Feb 28, 2020

@BPeuch hmm, is the following issue related?

@LauraHuisintveld
Copy link

For users of DataverseNL, the 'Add Data' button in the navigation bar is very confusing. This button always leads to creating a dataverse or dataset in the root. However, with our set-up, there is not a single user who has the permission to create a dataset or dataverse in the root. (except for a superuser account).

So the following situation might happen. A user navigates to a subdataverse where he is allowed to add data. He clicks on the "add data" button in the navigation bar and receives an error message, because he is not allowed to add something to the root-dataverse. However, if the same user had clicked on the "add data" button that appears for him below the navigation bar (the older button), he is able to create a dataset.

Maybe it would be better if the button in the navigation bar would not always lead to adding something in the root, but to add data in the dataverse you are currently viewing on your screen?

@BPeuch
Copy link
Contributor Author

BPeuch commented Mar 2, 2020

It is related, @pdurbin, in that I agree with your idea of "encourag[ing] user engagement".

@LauraHuisintveld: Very much agreed. Thanks for bringing up this issue.

@poikilotherm
Copy link
Contributor

We discussed this just this morning as we had some users confused when trying to add data to the root dataverse of Jülich DATA (which is not allowed).

We were thinking maybe instead of very generic error message, the page could generate a list of dataverses the user actually has permission to add things as links, so the UX is not "getting stuck and doesn't work (TM)".

@poikilotherm poikilotherm added Feature: Accessibility Accessibility is the practice of making your websites usable by as many people as possible. Feature: Permissions Small UX & UI: Design This issue needs input on the design of the UI and from the product owner Type: Suggestion an idea labels Mar 2, 2020
@djbrooke
Copy link
Contributor

djbrooke commented Mar 2, 2020

Thanks all for the feedback. We're planning on working on the workflows for adding data in the near future, and this was a first step. It looks like we need to make some adjustments here. I'll check in with @TaniaSchlatter and other members of the design team for their thoughts about a short term display logic solution.

@poikilotherm I don't think we want to display all the dataverses that the user can add data as it could be a very long message. Please hold off on working on this until we get some guidance from the design team. Thanks!

@djbrooke djbrooke removed the Small label Mar 2, 2020
@poikilotherm
Copy link
Contributor

@djbrooke it could be easily truncated to let's say the top 10 or however. Pretty sure there are multitude of ways how to do this for a good UX. Confident @TaniaSchlatter et al will come up with sth. brilliant for all of this. 😄

@BPeuch
Copy link
Contributor Author

BPeuch commented Mar 2, 2020

@djbrooke wrote:

I don't think we want to display all the dataverses that the user can add data as it could be a very long message.

When I read that I thought "How bad can it be?" Then I thought I'd check a mature Dataverse, like Harvard's, and indeed… 😅

amount_dataverses

@kaitlinnewson
Copy link
Contributor

For Scholars Portal we unfortunately have to remove this 'Add Data' button before we upgrade to 4.19 as we never allow users to post in the root (similar to @LauraHuisintveld). Shouldn't this only appear if they have permissions to publish in root, or be applied to the specific Dataverse that the user is currently in? For our context, we want users to publish in their specific dataverse for their institution, but of course this varies by installation.

@djbrooke
Copy link
Contributor

I'm going to close this as a duplicate of #6741, where we've started to evaluate how we can change display logic to support these cases.

@TaniaSchlatter
Copy link
Member

Also related to the solution described in #6772

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature: Accessibility Accessibility is the practice of making your websites usable by as many people as possible. Feature: Permissions Type: Suggestion an idea UX & UI: Design This issue needs input on the design of the UI and from the product owner
Projects
None yet
Development

No branches or pull requests

7 participants