Redmine

any news?

+1 It would be very good to get this, as we have to hide our estimations from some roles :(

+1 on this one!

+1

is covered by #8050

made a plugin for this: http://www.redmine.org/plugins/redmine_hide_estimated_hours

does anybody have version 1.0.0 of this plugin?

+1 It would be great to have estimated times integrated in the allover times permissions system.

+1

Now the patch cannot be applied cleanly.

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.

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