summaryrefslogtreecommitdiff
path: root/app/observers/note_observer.rb
Commit message (Collapse)AuthorAgeFilesLines
* Remove NotesObserverDmitriy Zaporozhets2014-06-171-20/+0
|
* Wipe wall notes featureDmitriy Zaporozhets2014-06-131-4/+1
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Remove thread vars usage from API notes and mr'sDmitriy Zaporozhets2014-06-101-2/+2
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Use EventCreateService for notes, merge requestsDmitriy Zaporozhets2014-03-251-0/+6
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Link issues from comments and automatically close themash wilson2013-08-251-0/+12
| | | | | | | | | | Any mention of Issues, MergeRequests, or Commits via GitLab-flavored markdown references in descriptions, titles, or attached Notes creates a back-reference Note that links to the original referencer. Furthermore, pushing commits with commit messages that match a (configurable) regexp to a project's default branch will close any issues mentioned by GFM in the matched closing phrase. If accepting a merge request would close any Issues in this way, a banner is appended to the merge request's main panel to indicate this.
* refactor observers. Added BaseObserver with helper methodsDmitriy Zaporozhets2013-03-261-7/+1
|
* Move new_note email logic to NotificationServiceDmitriy Zaporozhets2013-03-261-32/+1
|
* Use NotificationService for observers pt1Dmitriy Zaporozhets2013-03-261-0/+4
|
* correct indentation in activity observerDmitriy Zaporozhets2013-02-251-1/+0
|
* fix app crash if author is missingDmitriy Zaporozhets2013-02-011-1/+3
|
* Fix sending commit note email to id instead emailDmitriy Zaporozhets2013-02-011-1/+1
|
* Merge commit 'master' into discussionsRiyad Preukschas2013-01-151-2/+2
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: app/assets/stylesheets/sections/notes.scss app/contexts/notes/load_context.rb app/models/project.rb app/observers/note_observer.rb app/roles/votes.rb app/views/commit/show.html.haml app/views/merge_requests/_show.html.haml app/views/merge_requests/diffs.js.haml app/views/merge_requests/show.js.haml app/views/notes/_note.html.haml features/steps/project/project_merge_requests.rb spec/models/note_spec.rb
| * email via sidekiq. start and stop rake tasksDmitriy Zaporozhets2013-01-091-2/+2
| |
* | Fix commit note notificationRiyad Preukschas2013-01-021-1/+1
|/
* fix notify_team on wallJohannes Schleifenbaum2012-12-071-1/+1
|
* Rename NoteObserver methods and clarify thingsRiyad Preukschas2012-10-181-8/+13
|
* Fix notification for notes on a wall.Robb Kidd2012-10-131-1/+2
| | | | | The fact that Notes on Walls have a noteable_type value of "nil" got lost in the refactoring of the observer.
* Reduce complexity: replace case statement with method lookup.Robb Kidd2012-10-111-10/+5
|
* Separate observing of Note and MergeRequestsRobb Kidd2012-10-101-0/+36
* Move is_assigned? and is_being_xx? methods to IssueCommonality This is behavior merge requests have in common with issues. Moved methods to IssueCommonality role. Put specs directly into merge_request_spec because setup differs for issues and MRs specifically in the "closed" factory to use. * Add MergeRequestObserver. Parallels IssueObserver in almost every way. Ripe for refactoring. * Rename MailerObserver to NoteObserver With merge request observing moved out of MailerObserver, all that was left was Note logic. Renamed to NoteObserver, added tests and updated application config for new observer names. Refactored NoteObserver to use the note's author and not rely on current_user. * Set current_user for MergeRequestObserver IssueObserver and MergeRequestObserver are the only observers that need a reference to the current_user that they cannot look up on the objects they are observing.
View Lorry Controller Status