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

task assignment for admins #2728

Merged
merged 3 commits into from
Jun 12, 2018
Merged

task assignment for admins #2728

merged 3 commits into from
Jun 12, 2018

Conversation

youri-k
Copy link
Contributor

@youri-k youri-k commented Jun 11, 2018

  • also fixes the project readAccesQ

Mailable description of changes:

  • admins can now get tasks from all projects in their organization

Steps to test:

  • create a project and task for a team other than the admin's team
  • try to get a task as the admin
  • you should be able to get a task from that project

Issues:


  • Ready for review

@youri-k youri-k self-assigned this Jun 11, 2018
@youri-k youri-k requested a review from fm3 June 11, 2018 13:52
@fm3
Copy link
Member

fm3 commented Jun 12, 2018

@youri-k do you know why the e2e-tests failed? Do we just need new snapshots or did something break?

@youri-k
Copy link
Contributor Author

youri-k commented Jun 12, 2018

I think we need new snapshots because the admin user now get's a project which doesn't belong to a team where he is a member but this is intended behavior now

@fm3 fm3 merged commit 7ff01ba into master Jun 12, 2018
@fm3 fm3 deleted the task-assignment-admins branch June 12, 2018 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

admins should get tasks from projects of all teams
2 participants