-
Notifications
You must be signed in to change notification settings - Fork 448
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
Allow Journal Managers to invite users to adopt a role - Editors, Authors, Readers, etc #9658
Comments
hi Devika, We talked about the validation errors in the forms today! The general technical idea behind this is that we have the REST API endpoint that gives us a list of errors, and the forms in a specific step select from a custom list of fields that they need to display. I have gone through the designs and we seem to have one way of displaying errors or information, when the step has failed. Would you be suggesting a similar display for validation errors or may be form-field level validation errors for those specific errors, such es entered email format is not correct? |
Hello Dulip, I'll share the designs for validation errors with you and Ipula shortly. My basic logic is to provide inline error validation and not when the user clicks "Next" or comes on the "review screen" Are there a list of specific errors you are looking at? If yes, then it will be easy for me to provide the designs keeping those specific errors in mind. |
Thanks Devika,
yes, I was meaning the inline errors as you anticipated. Actually, the set of errors, I was looking at was the non-occurance of an input text or the validation according to a certail rule . A message e.g. for missing email and the sytnax validation would be an example. I tested the ui-library for some of the developments from Jarda and Ipula today and saw some of the validation error messages are still available there. May be they are from the old code still. |
@Devika008 I would like to specifically check with you, whether the fields in table (like selecting start date for role) would also show it inline the same way as any other form fields? |
It would be great if the errors on the table were inline so to replicate the behaviour elsewhere on the system and ease the cognitive load and learning graph of the user however I leave it to you if you think it would be difficult in the table then we can discuss |
* send and accept invitation * improved creating invitation workflow structure * send invitation * Clean Documentation * Change ORCiD Id to ORCID id * remove modal --------- Co-authored-by: Jarda Kotěšovec <[email protected]> Co-authored-by: withanage <[email protected]>
…ith text inputs on invitation pages
… inputs on invitation pages
…ith text inputs on invitation pages (#377)
…puts on invitation pages (#10156)
@Devika008 @jardakotesovec Can this ticket plus #3022 now be closed given that #9660 and #9661 are in progress? |
Hello,
Since Allow journal managers to invite users to adopt a role was too big, upon discussion we decided to divide it into four smaller issues, the first one being this.
This issue highlights only the user flow of journal managers inviting users like editors, authors, readers, etc to OJS. The user flow reflects all the research, discussions and feedback received on this issue over the last few years along with some minor testing with a small user group at the Copenhagen Sprint.
Summary of the requirement
Multiple hosted Journal OJS Installation: For Multiple Hosted Journals, Editors/ JMs cannot access information of users of one Journal to another. In spite of the journals being hosted on the same OJS instance, the user will have to go through the verification again to protect the information to be GDPR compliant. This is the same method followed by likes of Slack wherein even if the channel is hosted by the same company, the invitation process needs to be followed and accepted by the user. However, I as a user will not have to create a new account, I can use my existing OJS logins and verify and accept the role.
Other important considerations
Below are the user flows for inviting users to a role in OJS
A. Landing Page ( Settings > Users & Roles > Users)
I have tweaked the interface to make it more accessible, instead of the arrows at the start of each row, I have added a "more options" icon at the end. I have shown three states of it below.
B. The JM/editor needs to add a new role to a known user and searches for the user via their username or email ID by clicking on “invite user to take a role
If an invitation is sent to an existing user to acquire a new role then the users name will appear in both the tables and not just one table
How the roles table functions
C. The JM/editor needs to add a new role to a known user and finds the user in the list or searches for the user via their username or email ID.
If the user goes ahead with the ORCID Verification
Incase the user decides to skip ORCID verification, the following screen will come
D. The editor needs to add a new user and the user doesn’t have any existing OJS Account
6.a
Incase the username already exists, the following indication/error message will appear
6.b.
E. For Multiple Hosted Journals, Editors/ JMs cannot access information of users of one Journal to another. In spite of the journals being hosted on the same OJS instance, the user will have to go through the verification again to protect their information and fort he process to be GDPR compliant. This is the same method followed by likes of Slack wherein even if the channel is hosted by the same company, the invitation process needs to be followed and accepted by the user. However, I as a user will not have to create a new account, I can use my existing OJS logins and verify and accept the role
An existing OJS user, has two options here, either to link their existing OJS account or to create a new one using a new email id. Do note that even if the user uses an existing account, no information about the user activity can be shared with other journals in the same installation
The text was updated successfully, but these errors were encountered: