summaryrefslogtreecommitdiff
path: root/app/services/system_note_service.rb
Commit message (Collapse)AuthorAgeFilesLines
* Use SELECT 1, instead SELECT COUNT(*) to ask for notes existencycleanup-mentionable-methods-argumentsPaco Guzman2016-10-041-1/+1
|
* Enable Lint/StringConversionInInterpolation cop and autocorrect offensesrs-string-conversion-in-interpolationRobert Speicher2016-10-031-1/+1
|
* Refactor url_helpers for system note service and remove duplication of logic ↵Matthew Dodds2016-09-301-5/+8
| | | | in spec
* Add link to comparison from system note, update changelogMatthew Dodds2016-09-291-1/+14
|
* Capitalize mentioned issue timeline notesClement Ho2016-08-251-5/+5
|
* Merge branch 'master' into diff-line-comment-vuejsDouwe Maan2016-08-031-76/+78
|\ | | | | | | | | | | # Conflicts: # app/models/discussion.rb # db/schema.rb
| * Refactor system notes service to make it singletonGrzegorz Bizon2016-07-191-76/+78
| |
* | Add 'Resolved all discussions' system noteDouwe Maan2016-07-281-0/+6
|/
* Services: code style fixes, minor refactoringservices_refactoring1Valery Sizov2016-07-061-3/+4
|
* Enable Style/EmptyLines cop, remove redundant onesrubocop/enable-cops-for-empty-linesGrzegorz Bizon2016-07-011-1/+0
|
* Merge branch 'adambutler/gitlab-ce-feature/support-diff-of-issue-title-rename'Douwe Maan2016-05-201-1/+8
|\ | | | | | | | | # Conflicts: # app/services/system_note_service.rb
| * Create DiffFilter and change SystemNoteService#change_title to use ↵Adam Butler2016-05-181-1/+8
| | | | | | | | Gitlab::Diff::InlineDiff
* | Move #create_confidentiality_note to Issues::UpdateServicemooreniemi/gitlab-ce-issue_15236Rémy Coutable2016-05-181-12/+7
| | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | Changing the confidentiality of an issue now creates a new system noteAlex Moore-Niemi2016-05-181-0/+19
|/ | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Escape HTML in commit titles in system note messagesStan Hu2016-05-071-1/+5
| | | | Closes #17348
* Start with iid on branch creationZeger-Jan van de Weg2016-04-141-1/+1
|
* Added & use Gitlab::Routing for URL helpersroutingYorick Peterse2016-04-011-1/+1
| | | | | | | Rails' "url_helpers" method creates an anonymous Module (which a bunch of methods) on every call. By caching the output of this method in a dedicated method we can shave off about 10 seconds of loading time for an issue with around 200 comments.
* Merge branch 'master' into feature/issue-moveGrzegorz Bizon2016-03-201-1/+13
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * master: Fix bug where wrong commit ID was being used in a merge request diff to show old image Remove CHANGELOG item that was added during merge resolution Improve the "easy WIP & un-WIP from link" feature Fix specs \#to_branch_name now uses the iid as postfix Add label description in tooltip to labels in issue index and sidebar Easily (un)mark merge request as WIP using link Use specialized system notes when MR is (un)marked as WIP another attempt to fix oauth issue attempting to fix omniauth problem Conflicts: app/assets/javascripts/issuable_form.js.coffee
| * Merge branch 'remove-wip' into 'master' Jacob Schatz2016-03-191-0/+12
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Easily (un)mark merge request as WIP using link Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/3768 and https://gitlab.com/gitlab-org/gitlab-ce/issues/3516 ## Link to add `WIP` prefix (underline is visible because of hover) ![wipless_title](/uploads/72a6f7119ba9d8043ca8329641e97c3b/wipless_title.png) ## Link to remove `WIP` prefix ![wip_title](/uploads/8620ad65da9ef620b180603520fead55/wip_title.png) ## System note after WIP is added ![wip_sysnote](/uploads/2de073b75e854d2c9e243eb8b5d5c259/wip_sysnote.png) ## Widget with link to remove WIP ![wip_widget](/uploads/cf83ea93743c4c26d9df759c17cb9d7b/wip_widget.png) ## Flash after WIP is removed ![wip_flash](/uploads/27b7240cd5d7ceeb8b7b477abd94d7ff/wip_flash.png) ## System note after WIP is removed ![wipless_sysnote](/uploads/c0d3368abdf21a2f253532a9a9594d90/wipless_sysnote.png) ## Widget when current user cannot remove the WIP prefix ![wip_widget_unauthorized](/uploads/174ccf1674be86dc81c3078fe297acb7/wip_widget_unauthorized.png) cc @creamzy See merge request !3006
| | * Merge remote-tracking branch 'origin/master' into remove-wipRémy Coutable2016-03-181-1/+13
| | |\
| | * | Improve the "easy WIP & un-WIP from link" featureRémy Coutable2016-03-181-2/+2
| | | |
| | * | Use specialized system notes when MR is (un)marked as WIPDouwe Maan2016-02-261-0/+12
| | | |
| * | | \#to_branch_name now uses the iid as postfixissue-branch-iid-postfixZeger-Jan van de Weg2016-03-181-1/+1
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Given the branch name 'mep-mep' with an iid being 1, the current way, master's way, would yield a branch name of 1-mep-mep. The problem for larger projects however would be that a developer might forget what iid the issue was. When this developer would try to tab complete it would: - Or result in 20+ branches possibly - Or start with the wrong digit, try again with digit++ - Would see 20 branches, repeat Thus the obvious way of solving this is letting the dev tab complete on the issue title, which is easier to remember.
* | | Merge branch 'master' into feature/issue-moveGrzegorz Bizon2016-03-191-0/+12
|\ \ \ | |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * master: (121 commits) Dedupe labels in labels selector in Dashboard pages Refactor colors and lists Add a safeguard in MergeRequest#compute_diverged_commits_count Fix an issue when the target branch of a MR had been deleted Add avatar to issue and MR pages header Cleanup somce css colors Re-group scss variables Refactor `Todo#target` Fixes issue with filter label missing on labels & milestones Rename `Todo#to_reference` to `Todo#target_reference` Fixed failing tests Updated controller with before_action Fixed other issues based on feedback Fixes issue on dashboard issues Full labels data in JSON Fixed issue with labels dropdown getting wrong labels Update CHANGELOG Use `Note#for_project_snippet?` to skip notes on project snippet Use `Commit#short_id` instead of `Commit.truncate_sha` Reuse `for_commit?` on conditional validations Update schema info comment on todo related files ... Conflicts: app/models/issue.rb db/schema.rb spec/models/issue_spec.rb
| * | Incorporate the review and update specZeger-Jan van de Weg2016-03-151-2/+2
| | | | | | | | | | | | The feature spec now also tests the absence of the new branch button
| * | Incorporate reviewZeger-Jan van de Weg2016-03-151-1/+1
| | |
| * | Link in the note when started a new branch from an issueZeger-Jan van de Weg2016-03-151-1/+4
| | |
| * | Enhance new branch button on an issueZeger-Jan van de Weg2016-03-151-0/+9
| | |
* | | Add minor improvements in code related to issue moveGrzegorz Bizon2016-03-171-6/+4
| | |
* | | Improve system notes that are added when issue is movedGrzegorz Bizon2016-03-171-9/+16
| | |
* | | Add new system note used when issue has been movedGrzegorz Bizon2016-03-171-0/+17
|/ /
* | Add support for keyword arguments in label reference methodGrzegorz Bizon2016-03-031-1/+1
|/
* updated system note service and spec based on feedbackJames Lopez2016-02-161-1/+3
|
* woo hoo - getting there with the cross-reference issue. Should fix the ↵James Lopez2016-02-121-4/+11
| | | | problem and updated spec
* WIP - fix and spec for cross reference issue with forksJames Lopez2016-02-041-2/+2
|
* Use `to_reference` in system notes.Douwe Maan2015-12-241-1/+1
|
* Link to milestone in "Milestone changed" system noteDouwe Maan2015-12-241-2/+2
|
* Backport JIRA serviceDrew Blessing2015-12-181-2/+7
|
* Satisfy Douwe MaanZeger-Jan van de Weg2015-12-071-1/+1
|
* Merge branch 'master' into merge-if-greenZeger-Jan van de Weg2015-12-071-1/+19
|\
| * Proper cross-project references when MR is closed by issueDouwe Maan2015-11-301-1/+1
| |
| * Monkey patching TaskList::Item is no longer required. #2296Rubén Dávila2015-11-201-1/+2
| |
| * Generate system note after Task item has been updated on Issue or Merge ↵Ruben Davila2015-11-191-0/+17
| | | | | | | | | | | | | | | | | | | | | | | | Request. #2296 Everytime the User check or uncheck a Task Item from the Issue or Merge Request description, a new update is going to be added to the activity logs of the Issue or Merge Request. Note that when using the edit form, you can only update the Task item status or add/delete/modify existing ones. Doing both actions is not fully supported.
* | Button fixZeger-Jan van de Weg2015-12-071-1/+1
| |
* | Incorporate feedbackZeger-Jan van de Weg2015-12-051-3/+3
| |
* | Specs for 'Merge When Build Succeeds'Zeger-Jan van de Weg2015-12-021-1/+1
| |
* | Refactor MergeWhenBuildSucceedsService and incorporate feedbackZeger-Jan van de Weg2015-11-231-2/+2
| |
* | API support, incorporated feedbackZeger-Jan van de Weg2015-11-181-1/+1
| |
* | Merge when build succeedsZeger-Jan van de Weg2015-11-021-0/+14
|/
* Clean up MR refresh service somewhat.dm-issue-3140Douwe Maan2015-10-211-1/+1
|