Feature #3088
Estimated hours field able to hide role based
Status: | New | Start date: | 2009-04-01 | |
---|---|---|---|---|
Priority: | Normal | Due date: | ||
Assignee: | - | % Done: | 0% | |
Category: | Permissions and roles | |||
Target version: | Candidate for next major release | |||
Resolution: |
Description
adding the option to hide this field to the "roles and permissions" manager
Related issues
History
#1
Updated by Eric Davis about 13 years ago
- Status changed from New to 7
- Assignee set to Eric Davis
- Estimated time deleted (
0.50)
I've got something like this hacked onto my site right now that I can extract and cleanup a bit.
#2
Updated by Tim Klein about 13 years ago
any news?
#3
Updated by Eric Davis over 11 years ago
- Assignee deleted (
Eric Davis)
#4
Updated by Anton Nepomnyaschih about 11 years ago
+1 It would be very good to get this, as we have to hide our estimations from some roles :(
#5
Updated by Marc Lindsay almost 11 years ago
+1 on this one!
#6
Updated by Nilson Uehara over 10 years ago
+1
#7
Updated by Terence Mill over 10 years ago
is covered by #8050
#8
Updated by Dominique Lederer over 10 years ago
made a plugin for this: http://www.redmine.org/plugins/redmine_hide_estimated_hours
#9
Updated by Pavel Bahushevich over 9 years ago
does anybody have version 1.0.0 of this plugin?
#10
Updated by Jean-Philippe Lang over 9 years ago
- Status changed from 7 to New
Assigned issue with no assignee back to New status.
#11
Updated by Go MAEDA over 3 years ago
- Duplicated by Feature #3781: Add permission to control viewing of Estimated Hours based on Role added
#12
Updated by Go MAEDA over 3 years ago
- Duplicated by Feature #15107: Add a permission option to view estimated hours added
#13
Updated by Marius BALTEANU over 3 years ago
- Related to Feature #12005: Mightful workflow field enhancement: hide added
#14
Updated by Go MAEDA over 3 years ago
- Duplicated by Feature #11963: Hide the estimated times on issues for certain roles added
#15
Updated by Bernhard Rohloff over 3 years ago
+1 It would be great to have estimated times integrated in the allover times permissions system.
#16
Updated by Thierry Brochu about 3 years ago
+1
#17
Updated by Yuichi HARADA about 3 years ago
- File add_permission_view_estimated_hours.png added
- File 3088.patch
added
Add "View estimated hours" permission to the "Roles and permissions".
If the project has the "Time tracking" module enabled and the role has "View estimated time" permission enabled, the "Estimated hours" field is available.
I attached a patch.
#18
Updated by Go MAEDA over 2 years ago
- Target version set to Candidate for next major release
#19
Updated by Go MAEDA over 2 years ago
Now the patch cannot be applied cleanly.
#20
Updated by Yuichi HARADA over 2 years ago
- File 3088-v2.patch
added
Go MAEDA wrote:
Now the patch cannot be applied cleanly.
Fixed the patch to work on the current trunk(r19478).
#21
Updated by Go MAEDA about 2 years ago
Yuichi HARADA wrote:
Fixed the patch to work on the current trunk(r19478).
Thank you for the patch. But I am hesitant to make this patch a candidate for 4.2.0. This is because existing users who can currently see estimated hours will lose access to it after applying the patch because view_estimated_hours will not be granted to no roles by default.
#22
Updated by Go MAEDA 2 months ago
Go MAEDA wrote:
Yuichi HARADA wrote:
Fixed the patch to work on the current trunk(r19478).
Thank you for the patch. But I am hesitant to make this patch a candidate for 4.2.0. This is because existing users who can currently see estimated hours will lose access to it after applying the patch because view_estimated_hours will not be granted to no roles by default.
Adding a migration like the following will resolve the above problem.
diff --git a/db/migrate/20220315101502_add_view_estimated_hours_to_all_existing_roles.rb b/db/migrate/20220315101502_add_view_estimated_hours_to_all_existing_roles.rb
new file mode 100644
index 000000000..ed2ad47de
--- /dev/null
+++ b/db/migrate/20220315101502_add_view_estimated_hours_to_all_existing_roles.rb
@@ -0,0 +1,9 @@
+class AddViewEstimatedHoursToAllExistingRoles < ActiveRecord::Migration[6.1]
+ def up
+ Role.all.each { |role| role.add_permission! :view_estimated_hours }
+ end
+
+ def down
+ # nothing to revert
+ end
+end