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.
0%
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
Updated by Mizuki ISHIKAWA over 4 years ago
- File feature-33174.patch feature-33174.patch added
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.
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.
Updated by Go MAEDA over 4 years ago
- Related to Feature #12794: Member groups not listed in project page added
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.
Updated by Mizuki ISHIKAWA over 4 years ago
- File feature-33174-with-deprecation-warning.patch feature-33174-with-deprecation-warning.patch added
Go MAEDA wrote:
Not to break third-party plugins, I think it is safer to keep
users_by_rolemethod 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.
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.
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.
Updated by Marius BĂLTEANU over 4 years ago
- File members.png members.png added
- Status changed from Closed to Reopened
Is it ok to show the builtin groups ("Anonymous users", "Non members users") and group members ("User misc" is member of A Team)?
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
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!