Project

General

Profile

Actions

Feature #33174

closed

Show groups in members box on project overview page

Added by Mizuki ISHIKAWA over 4 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Projects
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed

Description

The project overview page shows only users, even if the group has been added as a member of the project.
The "Members" tab of the project settings shows both users and groups, so I think the overview page needs to show groups as well as users.


Files

ScreenShot-2020-03-19-10.32.49.png (245 KB) ScreenShot-2020-03-19-10.32.49.png Mizuki ISHIKAWA, 2020-03-19 02:35
feature-33174.patch (3.02 KB) feature-33174.patch Mizuki ISHIKAWA, 2020-03-19 02:36
feature-33174-with-deprecation-warning.patch (3.34 KB) feature-33174-with-deprecation-warning.patch Mizuki ISHIKAWA, 2020-04-01 06:13
members.png (50.4 KB) members.png Marius BĂLTEANU, 2020-04-24 22:59

Related issues

Related to Redmine - Feature #12794: Member groups not listed in project pageClosed

Actions
Actions #2

Updated by Yuuki NARA over 4 years ago

+1

This ticket is related to #12794.

I have a suggestion.

If more than 100 people are set in the group definition,
Displaying only the group definition name on the overview screen makes it easier to understand.

Many people may be expanded and displayed due to inheritance from the upper-level PJ and expansion of the group definition.

Actions #3

Updated by Go MAEDA over 4 years ago

Yuuki NARA wrote:

I have a suggestion.

If more than 100 people are set in the group definition,
Displaying only the group definition name on the overview screen makes it easier to understand.

I think it is another feature. Maybe it is better to discuss the additional feature after the patch is accepted. Generally, expanding the scope of an issue makes it difficult to move it forward.

Actions #4

Updated by Go MAEDA over 4 years ago

  • Related to Feature #12794: Member groups not listed in project page added
Actions #5

Updated by Go MAEDA over 4 years ago

  • Target version set to Candidate for next major release

Not to break third-party plugins, I think it is safer to keep users_by_role method and log a deprecation warning like source:tags/3.4.9/app/models/project.rb#L839 than removing the method in Redmine 4.2.0.

Actions #6

Updated by Mizuki ISHIKAWA over 4 years ago

Go MAEDA wrote:

Not to break third-party plugins, I think it is safer to keep users_by_role method and log a deprecation warning like source:tags/3.4.9/app/models/project.rb#L839 than removing the method in Redmine 4.2.0.

Changed Project#members_by_role to give deprecation warning.

Actions #7

Updated by Go MAEDA over 4 years ago

  • Target version changed from Candidate for next major release to 4.2.0

Setting the target version to 4.2.0.

Actions #8

Updated by Go MAEDA over 4 years ago

  • Status changed from New to Closed
  • Assignee set to Go MAEDA
  • Resolution set to Fixed

Committed the patch. Thank you for improving Redmine.

Actions #9

Updated by Marius BĂLTEANU over 4 years ago

Is it ok to show the builtin groups ("Anonymous users", "Non members users") and group members ("User misc" is member of A Team)?

Actions #10

Updated by Go MAEDA over 4 years ago

Marius BALTEANU wrote:

Is it ok to show the builtin groups ("Anonymous users", "Non members users") and group members ("User misc" is member of A Team)?

I think it is how it should behave.

  • The list is used to see the members of projects, so builtin groups should be displayed as well if they are a member
  • Users without special permission have no means to know group members, so I think users added to the project through group should be displayed as well. The members tab in the project setting also shows group members
Actions #11

Updated by Marius BĂLTEANU over 4 years ago

  • Status changed from Reopened to Closed

Go MAEDA wrote:

I think it is how it should behave.

Thanks!

Actions

Also available in: Atom PDF