summaryrefslogtreecommitdiff
path: root/app/services/issues
Commit message (Collapse)AuthorAgeFilesLines
* Check if user can read issue before being assignedissue_22664Felipe Artur2016-12-271-4/+0
|
* Enable Style/MultilineOperationIndentation in Rubocop, fixes #25741Rydkin Maxim2016-12-161-1/+1
|
* Feature: delegate all open discussions to IssueBob Van Landuyt2016-12-053-1/+71
| | | | | | | | | | | | | When a merge request can only be merged when all discussions are resolved. This feature allows to easily delegate those discussions to a new issue, while marking them as resolved in the merge request. The user is presented with a new issue, prepared with mentions of all unresolved discussions, including the first unresolved note of the discussion, time and link to the note. When the issue is created, the discussions in the merge request will get a system note directing the user to the newly created issue.
* Refactor JiraService by moving code out of JiraService#execute methodclean-up-jira-serviceAdam Niedzielski2016-12-011-1/+1
| | | | | | | | | | The implicit interface of project services states that the "execute" method is meant to be called when project hooks are executed. Currently JiraService does not support any project events even though JiraService#supported_events says that "commit" and "merge_request" are supported. They are only used to render correct options in JIRA configuration screen, but they are not supported. Because of that, this commit makes "execute" method a no-op.
* Process commits in a separate workerprocess-commits-using-sidekiqYorick Peterse2016-11-071-0/+13
| | | | | | | | | | | | | This moves the code used for processing commits from GitPushService to its own Sidekiq worker: ProcessCommitWorker. Using a Sidekiq worker allows us to process multiple commits in parallel. This in turn will lead to issues being closed faster and cross references being created faster. Furthermore by isolating this code into a separate class it's easier to test and maintain the code. The new worker also ensures it can efficiently check which issues can be closed, without having to run numerous SQL queries for every issue.
* Include global labels when moving an issue to another projectDouglas Barbosa Alexandre2016-10-191-2/+6
|
* Add bulk update support for merge requests listDmitriy Zaporozhets2016-09-081-25/+0
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Scope hooks thal will run for confidential issuesDouglas Barbosa Alexandre2016-08-311-5/+4
|
* Fix confidential issues should not be passed to WebhooksDouglas Barbosa Alexandre2016-08-311-0/+2
|
* Merge branch '4273-slash-commands' into 'master'Robert Speicher2016-08-183-15/+16
|\ | | | | | | | | | | Support slash commands in issues / MR description & comments See merge request !5021
| * Fix specs and implement fixes based on failing specsDouwe Maan2016-08-171-2/+5
| |
| * Merge branch 'master' into 4273-slash-commandsDouwe Maan2016-08-161-7/+12
| |\ | | | | | | | | | | | | # Conflicts: # app/services/issues/create_service.rb
| * | Enforce permissions in `{Issues,MergeRequests}::{Close,Reopen}Service`Rémy Coutable2016-08-132-0/+4
| | | | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
| * | Support slash commands in noteable description and notesRémy Coutable2016-08-131-13/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some important things to note: - commands are removed from noteable.description / note.note - commands are translated to params so that they are treated as normal params in noteable Creation services - the logic is not in the models but in the Creation services, which is the right place for advanced logic that has nothing to do with what models should be responsible of! - UI/JS needs to be updated to handle notes which consist of commands only - the `/merge` command is not handled yet Other improvements: - Don't process commands in commit notes and display a flash is note is only commands - Add autocomplete for slash commands - Add description and params to slash command DSL methods - Ensure replying by email with a commands-only note works - Use :subscription_event instead of calling noteable.subscribe - Support :todo_event in IssuableBaseService Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | Merge branch '2451-fix-mentions-in-issue-updates' into 'master' Robert Speicher2016-08-171-1/+6
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Send notification emails when users are newly mentioned in issue or MR edits ## What does this MR do? Introduces "new mention in issue" and "new mention in MR" email notifications. Editing a Mentionable title or description and adding a mention to a user who was not previously mentioned will now send them a notification email, following usual permissions for doing so. ## Why was this MR needed? Issues & MRs may be edited to include mentions to new people. We don't currently send out email notifications of these edits to anyone, although they do create TODOs. This brings email notifications into parity with TODOs. ## What are the relevant issue numbers? Closes #2451 See merge request !5800
| * | Send notification emails when users are newly mentioned in issue editsNick Thomas2016-08-121-1/+6
| |/
* | Further refactor and syntax fixes.Patricio Cano2016-08-151-2/+2
| |
* | Refactored AkismetHelper into AkismetService and cleaned up `Spammable`Patricio Cano2016-08-151-3/+3
| | | | | | | | - Refactored SpamCheckService into SpamService
* | Refactored spam related code even furtherPatricio Cano2016-08-151-17/+17
| | | | | | | | | | | | | | - Removed unnecessary column from `SpamLog` - Moved creation of SpamLogs out of its own service and into SpamCheckService - Simplified code in SpamCheckService. - Moved move spam related code into Spammable concern
* | Complete refactor of the `Spammable` concern and tests:Patricio Cano2016-08-151-1/+1
| | | | | | | | | | | | | | - Merged `AkismetSubmittable` into `Spammable` - Clean up `SpamCheckService` - Added tests for `Spammable` - Added submit (ham or spam) options to `AkismetHelper`
* | Lay the ground works to submit information to AkismetPatricio Cano2016-08-151-0/+5
|/ | | | | | - New concern `AkismetSubmittable` to allow issues and other `Spammable` models to be submitted to Akismet. - New model `UserAgentDetail` to store information needed for Akismet. - Services needed for their creation and tests.
* Refactor spam validation to a concern that can be easily reused and improve ↵akismet-ui-checkPatricio Cano2016-07-261-6/+11
| | | | legibility in `SpamCheckService`
* Refactor `SpamCheckService` to make it cleaner and clearer.Patricio Cano2016-07-262-27/+3
|
* Submit all issues on public projects to Akismet if enabled.Patricio Cano2016-07-262-1/+30
|
* Allow bulk (un)subscription from issues in issue indexCairo Noleto2016-07-141-1/+1
| | | | fixies #19747
* Services: code style fixes, minor refactoringservices_refactoring1Valery Sizov2016-07-061-0/+1
|
* Enable Style/EmptyLines cop, remove redundant onesrubocop/enable-cops-for-empty-linesGrzegorz Bizon2016-07-011-1/+0
|
* Allow bulk-updating to remove all labelsSean McGivern2016-06-031-1/+1
| | | | | Instead of passing `remove_label_ids`, just pass an empty array for `label_ids` (and don't pass `add_label_ids` or `remove_label_ids`).
* WIP: allow adding and removing labels in bulkSean McGivern2016-06-031-3/+3
|
* :police_car:Z.J. van de Weg2016-06-031-6/+6
|
* Move awardables too when issue is movedZ.J. van de Weg2016-06-011-0/+9
|
* Move #create_confidentiality_note to Issues::UpdateServicemooreniemi/gitlab-ce-issue_15236Rémy Coutable2016-05-181-1/+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-1/+5
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Update changelog, improve specsLong Nguyen2016-05-051-1/+1
|
* Code refactor and fix broken specLong Nguyen2016-04-281-3/+5
|
* Allow to assign labels and milestone to target project when moving issueLong Nguyen2016-04-261-1/+10
|
* Refactor and expose only Gitlab::UrlBuilder.build(record)simplify-gitlab-url_builder-15202Rémy Coutable2016-04-131-1/+1
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Check if GFM rewriters need rewrite internallyGrzegorz Bizon2016-03-301-1/+1
|
* Refactor uploads rewriter used when moving issueGrzegorz Bizon2016-03-301-17/+6
|
* Add uploads rewriter and use it when moving issueGrzegorz Bizon2016-03-301-12/+27
|
* Preserve time notes has been updated at when moving issueGrzegorz Bizon2016-03-231-1/+2
|
* Do not allow to move issue if it has not been persistedmove-issue-section-should-not-be-displayed-in-the-new-issue-form-14489Grzegorz Bizon2016-03-231-0/+2
|
* Preserve created at time of notes when moving issueGrzegorz Bizon2016-03-201-1/+2
|
* Rename reference unfolder to rewriter, minor refactoringsGrzegorz Bizon2016-03-201-2/+2
|
* Update reference unfolder according to recent ability changesGrzegorz Bizon2016-03-191-1/+2
| | | | | Commit 43d8bdb4f048cbeb5675ed9120cb1aeb415b9586 introduced additional checks for permissions to read issue in references extractor.
* Improvements in issue move feaure (refactoring)Grzegorz Bizon2016-03-192-52/+39
| | | | According to endbosses' suggestions.
* Prevent issue move if issue has been already movedGrzegorz Bizon2016-03-171-2/+7
|
* Update methods that use issue close serviceGrzegorz Bizon2016-03-171-1/+1
| | | | | `Issues::CloseService#execute` signature has changed, because of using keyword parameter for commmit.
* Add minor improvements in code related to issue moveGrzegorz Bizon2016-03-171-20/+11
|
* Add new notifications for issue move actionGrzegorz Bizon2016-03-171-2/+7
| | | | [ci skip]
View Lorry Controller Status