Add alias_attributes for notes

app/models/note.rb

@@ -18,6 +18,9 @@ class Note < ActiveRecord::Base
   cache_markdown_field :note, pipeline: :note, issuable_state_filter_enabled: true
+  alias_attribute :last_edited_at, :updated_at
+  alias_attribute :last_edited_by, :updated_by
+
   # Attribute containing rendered and redacted Markdown as generated by
   # Banzai::ObjectRenderer.
   attr_accessor :redacted_note_html

app/services/notes/update_service.rb

@@ -5,11 +5,7 @@ module Notes
       old_mentioned_users = note.mentioned_users.to_a
-      note.assign_attributes(params)
-      params.merge!(last_edited_at: Time.now, last_edited_by: current_user) if note.note_changed?
-
       note.update_attributes(params.merge(updated_by: current_user))
-
       note.create_new_cross_references!(current_user)
       if note.previous_changes.include?('note')

db/migrate/20170503022512_add_last_edited_at_and_last_edited_by_id_to_notes.rb

-# See http://doc.gitlab.com/ce/development/migration_style_guide.html
-# for more information on how to write migrations for GitLab.
-
-class AddLastEditedAtAndLastEditedByIdToNotes < ActiveRecord::Migration
-  include Gitlab::Database::MigrationHelpers
-
-  # Set this constant to true if this migration requires downtime.
-  DOWNTIME = false
-
-  def change
-    add_column :notes, :last_edited_at, :timestamp
-    add_column :notes, :last_edited_by_id, :integer
-  end
-end

db/schema.rb

@@ -778,8 +778,6 @@ ActiveRecord::Schema.define(version: 20170503022548) do
     t.string "discussion_id"
     t.text "note_html"
     t.integer "cached_markdown_version"
-    t.datetime "last_edited_at"
-    t.integer "last_edited_by_id"
   end
   add_index "notes", ["author_id"], name: "index_notes_on_author_id", using: :btree

spec/services/notes/update_service_spec.rb

@@ -20,13 +20,6 @@ describe Notes::UpdateService, services: true do
       @note.reload
     end
-    it 'updates last_edited_at and last_edited_by attributes' do
-      update_note({ note: 'Hello world!' })
-
-      expect(@note.last_edited_at).not_to be_nil
-      expect(@note.last_edited_by).not_to be_nil
-    end
-
     context 'todos' do
       let!(:todo) { create(:todo, :assigned, user: user, project: project, target: issue, author: user2) }