ActionItem628: linking directly to the URL where pending tasks can be

solved.

ActionItem628: linking directly to the URL where pending tasks can be
solved.
Antonio Terceiro
1 parent 403fa5ef
Showing 4 changed files with 7 additions and 3 deletions Show diff stats
app/models/add_friend.rb
app/models/approve_article.rb
app/models/task_mailer.rb
app/views/task_mailer/target_notification.rhtml
@@ -27,7 +27,7 @@ class AddFriend &lt; Task
   def target_notification_message
     description + "\n\n" +
-    _('You need to login to %{system} in order to accept %{requestor} as your friend. You can use the address below to do that.') % { :system => target.environment.name, :requestor => requestor.name }
+    _('You need to login to %{system} in order to accept %{requestor} as your friend.') % { :system => target.environment.name, :requestor => requestor.name }
   end
 end
@@ -41,7 +41,7 @@ class ApproveArticle &lt; Task
   def target_notification_message
     description + "\n\n" +
-    _('You need to login on %{system} in order to approve or reject this article. You can use the address below to do that.') % { :system => target.environment.name }
+    _('You need to login on %{system} in order to approve or reject this article.') % { :system => target.environment.name }
   end
 end
@@ -20,7 +20,8 @@ class TaskMailer &lt; ActionMailer::Base
       :target => task.target.name,
       :message => msg,
       :environment => task.requestor.environment.name,
-      :url => url_for(:host => task.requestor.environment.default_hostname, :controller => 'home')
+      :url => url_for(:host => task.requestor.environment.default_hostname, :controller => 'home'),
+      :tasks_url => url_for(task.target.url.merge(:controller => 'tasks', :action => 'index'))
   end
   protected
@@ -2,6 +2,9 @@
 <%= word_wrap(@message) %>
+<%= word_wrap(_('Access the address below to see this and other pending actions that need your attention:')) %>
+<%= @tasks_url %>
+
 --
 <%= @environment %>
 <%= @url %>
	@@ -27,7 +27,7 @@ class AddFriend < Task		@@ -27,7 +27,7 @@ class AddFriend < Task
27		27
28	def target_notification_message	28	def target_notification_message
29	description + "\n\n" +	29	description + "\n\n" +
30	- _('You need to login to %{system} in order to accept %{requestor} as your friend. You can use the address below to do that.') % { :system => target.environment.name, :requestor => requestor.name }	30	+ _('You need to login to %{system} in order to accept %{requestor} as your friend.') % { :system => target.environment.name, :requestor => requestor.name }
31	end	31	end
32		32
33	end	33	end
	@@ -41,7 +41,7 @@ class ApproveArticle < Task		@@ -41,7 +41,7 @@ class ApproveArticle < Task
41		41
42	def target_notification_message	42	def target_notification_message
43	description + "\n\n" +	43	description + "\n\n" +
44	- _('You need to login on %{system} in order to approve or reject this article. You can use the address below to do that.') % { :system => target.environment.name }	44	+ _('You need to login on %{system} in order to approve or reject this article.') % { :system => target.environment.name }
45	end	45	end
46		46
47	end	47	end