hipster-philology / pyrrha

A language-independent post-correction app for POS-tagging and lemmatization
https://pyrrha.huma-num.fr
MIT License
27 stars 16 forks source link

As full administrator, I cannot propose changes to Control List Administators #135

Closed PonteIneptique closed 3 years ago

PonteIneptique commented 5 years ago

Current Behaviour

image

  1. As a non owner of the control list and as administrator
  2. For a Control List, click the Control List top menu, select a control list.
  3. On the left menu bar, there is no links (if the CL is not public), or there is a link to make it public

Expected behaviour

I should see the Propose Change link under other to send a mail to list administrator

carinedengler commented 4 years ago

We estimated this ticket to take 0.5d.

carinedengler commented 4 years ago

We do not seem to be able to reproduce this bug on either of our local instances, we tried

This seems to be the expected behavior for this link as described by you.

However, we always have at least two buttons on the left-hand side menu, one to rename the list and one to modify its related information, whereas the button to make it public (if it is private) is not in that menu at all, but in the controls view. Our instances are at the top of the dev branch.

MrGecko commented 4 years ago

The current behavior present in https://github.com/hipster-philology/pyrrha/blob/dev/app/templates/control_lists/macros.html is the following (content truncated for clarity purpose):

      {% if not control_list.can_edit() %}
      Propose changes
      {% endif %}

      {%  if control_list.can_edit() and not current_user.is_admin() %}
      Make public
      {% endif %}

      {%  if control_list.can_edit() or current_user.is_admin() %}
      Rename
      {% endif %}

      {%  if control_list.can_edit() or current_user.is_admin() %}
      Edit informations
      {% endif %}

Therefore if you can edit, you cannot propose change (thus you don't see the link).

@PonteIneptique I have a hard time to reproduce the "no link at all" case.

carinedengler commented 4 years ago

When I try to reproduce the issue, I have the following user interface

control-list-lemma-admin

In that case, I am admin, and it's a public control list. If it's a private control list of which, I, as admin, am not the owner, I have this user interface

control-list-pos-admin-private

In both cases, the link to propose changes is visible and clickable, and the control lists in question are editable. I am on the head of the dev branch, and haven't implemented any changes.

PonteIneptique commented 3 years ago

I think "Propose changes" should be always on. That will "fix" this issue.