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

ui: Suggest adding permissions when assigning a (sub-)project #23

Closed
phpai opened this issue Mar 22, 2019 · 2 comments · Fixed by #389
Closed

ui: Suggest adding permissions when assigning a (sub-)project #23

phpai opened this issue Mar 22, 2019 · 2 comments · Fixed by #389
Labels
feature Indicates a new feature implementation

Comments

@phpai
Copy link
Contributor

phpai commented Mar 22, 2019

In GitLab by @kevinbader on Jul 30, 2018, 10:40

Problem

When assigning someone to a project or subproject, the assignee is not granted any permissions. This should stay that way, but there is an unfortunate side effect: it may happen that an assignee doesn't even have permissions to view the project or subproject she's assigned to, which clearly is unexpected behaviour to the user.

Proposed Solution

Instead of adding any automatic permission changes to *.assign intents, the UI should check whether the current assignee has all expected permissions. If not, the UI should offer to change the assignee's permissions, using a modal dialog. If the current user is not allowed to grant additional permissions, the check doesn't need to be done and the dialog should not appear.

@phpai phpai added this to the TruBudget 1.1.0 milestone Mar 22, 2019
@phpai phpai added feature Indicates a new feature implementation low prio labels Mar 22, 2019
@phpai
Copy link
Contributor Author

phpai commented Mar 22, 2019

In GitLab by @kevinbader on Aug 10, 2018, 11:43

We'll need to tackle dialogs more broadly than this. For example, this will hardly remain the only "convenience" dialog we need, and there is also the need for "do you really want to do this" dialogs in various places.

-> rescoping to a later beta release of 1.0.0

@Stezido
Copy link
Contributor

Stezido commented May 29, 2019

Yes we should implement the permission checks on UI-level. As an addition to the idea of the modal dialog I think if the user which is assigning has no permissions to grant additional permissions at least a "do you really want to do this" dialog should open with warnings that the assigned user would not see the project she would be assigned to. Otherwise we have the same initial problem and unexpected behaviour.

I also agree with the broadly dialogs!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Indicates a new feature implementation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants