--- a/app/models/mailer.rb
+++ b/app/models/mailer.rb
@@ -21,6 +21,17 @@ class Mailer < ActionMailer::Base
     @body[:url] = "http://gitorious.org/p/#{repository.project.slug}/repos/#{repository.name}"
   end
   
+  def merge_request_notification(merge_request)
+    setup_email(merge_request.target_repository.user)
+    @subject += %Q{#{merge_request.target_repository.user} has requested a merge in #{merge_request.target_repository.project}}
+    @body[:merge_request] = merge_request
+    @body[:project] = merge_request.target_repository.project
+    url = "http://gitorious.org/p/#{merge_request.target_repository.project.slug}"
+    url << "/repos/#{merge_request.target_repository.project.slug}"
+    url << "/merge_requests/#{merge_request.id}"
+    @body[:url] = url
+  end
+  
   protected
     def setup_email(user)
       @recipients  = "#{user.email}"

--- a/app/models/merge_request.rb
+++ b/app/models/merge_request.rb
@@ -9,4 +9,16 @@ class MergeRequest < ActiveRecord::Base
   
   validates_presence_of :user, :source_repository, :target_repository
   attr_protected :user_id
+  
+  def open?
+    status == STATUS_OPEN
+  end
+  
+  def merged?
+    status == STATUS_MERGED
+  end
+  
+  def rejected?
+    status == STATUS_REJECTED
+  end
 end

--- /dev/null
+++ b/app/models/merge_request_observer.rb
@@ -0,0 +1,7 @@
+class MergeRequestObserver < ActiveRecord::Observer
+  
+  def after_create(record)
+    Mailer.deliver_merge_request_notification(record)
+  end
+  
+end
\ No newline at end of file

--- /dev/null
+++ b/app/views/mailer/merge_request_notification.rhtml
@@ -0,0 +1,10 @@
+Hello <%= @merge_request.target_repository.user.login -%>,
+
+<%= @merge_request.source_repository.user.login -%> has requested that you merge <%= @merge_request.source_repository.name -%> with <%= @merge_request.target_repository.name -%> in the <%= @project.title -%> project.
+
+You can review the request and its commits here:
+<%= @url %>
+
+Thank you
+
+http://gitorious.org

--- a/config/environment.rb
+++ b/config/environment.rb
@@ -45,7 +45,11 @@ Rails::Initializer.run do |config|
   # config.active_record.schema_format = :sql
 
   # Activate observers that should always be running
-  config.active_record.observers = [:user_observer, :repository_observer]
+  config.active_record.observers = [
+      :user_observer, 
+      :repository_observer, 
+      :merge_request_observer
+  ]
 
   # Make Active Record use UTC-base instead of local time
   config.active_record.default_timezone = :utc

--- a/spec/models/mailer_spec.rb
+++ b/spec/models/mailer_spec.rb
@@ -44,5 +44,18 @@ describe Mailer do
     Mailer.deliver(mail)
     Mailer.deliveries.should == [mail]
   end
+  
+  it "sends merge_request_notification" do
+    merge_request = merge_requests(:moes_to_johans)
+    mail = Mailer.create_merge_request_notification(merge_request)
+    
+    mail.to.should == [merge_request.target_repository.user.email]
+    mail.subject.should == "[Gitorious] #{merge_request.target_repository.user} has requested a merge in #{merge_request.target_repository.project}"
+    mail.body.should match(/moe has requested that you merge #{merge_request.source_repository.name} with #{merge_request.target_repository.name}/)
+    mail.body.should match(/in the #{merge_request.target_repository.project.title} project/)
+    
+    Mailer.deliver(mail)
+    Mailer.deliveries.should == [mail]
+  end
 
 end

--- a/spec/models/merge_request_spec.rb
+++ b/spec/models/merge_request_spec.rb
@@ -24,9 +24,25 @@ describe MergeRequest do
     @merge_request.should have(1).error_on(:target_repository)
   end
   
-  it "emails the owner of the target_repository on create"
+  it "emails the owner of the target_repository on create" do
+    Mailer.deliveries = []
+    mr = @merge_request.clone
+    mr.save
+    Mailer.deliveries.should_not be_empty
+  end
   
-  it "has a merged? status"
+  it "has a merged? status" do
+    @merge_request.status = MergeRequest::STATUS_MERGED
+    @merge_request.merged?.should == true
+  end
   
-  it "has a rejected? status"
+  it "has a rejected? status" do
+    @merge_request.status = MergeRequest::STATUS_REJECTED
+    @merge_request.rejected?.should == true
+  end
+  
+  it "has a open? status" do
+    @merge_request.status = MergeRequest::STATUS_OPEN
+    @merge_request.open?.should == true
+  end
 end