Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Add email for milestone change | Chantal Rollison | 2018-11-02 | 1 | -0/+14 |
| | |||||
* | Enable more frozen string in app/services/**/*.rbrepo-forks/gitlab-ce-frozen-string-enable-apps-services-inner-more | gfyoung | 2018-07-17 | 1 | -0/+2 |
| | | | | Partially addresses #47424. | ||||
* | Move NotificationService calls to Sidekiq | Sean McGivern | 2018-04-25 | 1 | -3/+8 |
| | | | | | | | | | | | | | | | | | | | | | The NotificationService has to do quite a lot of work to calculate the recipients for an email. Where possible, we should try to avoid doing this in an HTTP request, because the mail are sent by Sidekiq anyway, so there's no need to schedule those emails immediately. This commit creates a generic Sidekiq worker that uses Global ID to serialise and deserialise its arguments, then forwards them to the NotificationService. The NotificationService gains an `#async` method, so you can replace: notification_service.new_issue(issue, current_user) With: notification_service.async.new_issue(issue, current_user) And have everything else work as normal, except that calculating the recipients will be done by Sidekiq, which will then schedule further Sidekiq jobs to send each email. | ||||
* | /wip slash command on MR creation | Adam Pahlevi | 2018-03-01 | 1 | -11/+0 |
| | | | | | | | | | | | | change to symbol add complete changelog add test for /wip unwip as sym test for work in progress separate from issuable | ||||
* | Refactor the way we pass `old associations` to issuable's update services40226-refactor-the-issuable-s-webhooks-data-architecture | Rémy Coutable | 2017-11-24 | 1 | -2/+3 |
| | | | | Signed-off-by: Rémy Coutable <remy@rymai.me> | ||||
* | CE port of code changed for epicsjk-epic-changes-ce-port | Jarka Kadlecova | 2017-11-02 | 1 | -4/+6 |
| | |||||
* | Rename MergeRequest#async_merge to merge_async | Oswaldo Ferreira | 2017-08-28 | 1 | -1/+1 |
| | |||||
* | Track enqueued and ongoing MRs | Oswaldo Ferreira | 2017-08-28 | 1 | -1/+1 |
| | |||||
* | Rename "Slash commands" to "Quick actions" | Eric Eastwood | 2017-06-15 | 1 | -3/+3 |
| | | | | | | | | | | | | | Fix https://gitlab.com/gitlab-org/gitlab-ce/issues/27070 Deprecate "chat commands" in favor of "slash commands" We looked for things like: - `slash commmand` - `slash_command` - `slash-command` - `SlashCommand` | ||||
* | Backport of multiple_assignees_feature [ci skip] | Valery Sizov | 2017-05-04 | 1 | -1/+4 |
| | |||||
* | Create todos only for new mentions | Jarka Kadlecova | 2017-03-29 | 1 | -1/+1 |
| | |||||
* | support `/merge` slash comand for MRs | Jarka Kadlecova | 2017-01-11 | 1 | -0/+15 |
| | |||||
* | Enable Style/MultilineOperationIndentation in Rubocop, fixes #25741 | Rydkin Maxim | 2016-12-16 | 1 | -1/+1 |
| | |||||
* | Remove unnecessary require_relative calls from service classes | Semyon Pupkov | 2016-11-22 | 1 | -4/+0 |
| | | | | | Rails by default use autoload for all dirs from app folder. require_relative not needed. See ActiveSupport::Dependencies.autoload_paths | ||||
* | Maintain "force_remove_source_branch" options on Merge Request unless specified | Mark Fletcher | 2016-10-13 | 1 | -1/+4 |
| | |||||
* | Add a /wip slash command | Thomas Balthazar | 2016-10-03 | 1 | -1/+14 |
| | | | | It toggles the 'WIP' prefix in the MR title. | ||||
* | Use the `IssuableBaseService` lifecycle hooks to cache ↵ | Timothy Andrew | 2016-09-21 | 1 | -0/+4 |
| | | | | | | | `MergeRequestsClosingIssues` - Instead of overriding `create` and `update` in `MergeRequests::BaseService` - Get all merge request service specs passing | ||||
* | User can edit closed MR with deleted fork | Katarzyna Kobierska | 2016-08-30 | 1 | -0/+4 |
| | | | | | | | | | | | | | | | | | | | | | | Add test for closed MR without fork Add view test visibility of Reopen and Close buttons Fix controller tests and validation method Fix missing space Remove unused variables from test closed_without_fork? method refactoring Add information about missing fork When closed MR without fork can't edit target branch Tests for closed MR edit view Fix indentation and rebase, refactoring | ||||
* | Add notifications for new user mentions in merge requests | Nick Thomas | 2016-08-15 | 1 | -1/+8 |
| | |||||
* | Fix a 'missing keyword' error introduced in the last commit | Nick Thomas | 2016-08-15 | 1 | -1/+3 |
| | |||||
* | Allows MR authors to have the source branch removed when merging the MR | Jeroen Jacobs | 2016-05-20 | 1 | -0/+2 |
| | |||||
* | Improving the original label-subscribing implementation | Rémy Coutable | 2016-03-15 | 1 | -5/+9 |
| | | | | | | | 1. Make the "subscribed" text in Issuable sidebar reflect the labels subscription status 2. Current user mut be logged-in to toggle issue/MR/label subscription | ||||
* | Original implementation to allow users to subscribe to labels | Timothy Andrew | 2016-03-15 | 1 | -1/+6 |
| | | | | | | | | | | | | | | | | | | 1. Allow subscribing (the current user) to a label - Refactor the `Subscription` coffeescript class - The main change is that it accepts a container, and conducts all DOM queries within its scope. We need this because the labels page has multiple instances of `Subscription` on the same page. 2. Creating an issue or MR with labels notifies users subscribed to those labels - Label `has_many` subscribers through subscriptions. 3. Adding a label to an issue or MR notifies users subscribed to those labels - This only applies to subscribers of the label that has just been added, not all labels for the issue. | ||||
* | Rename Tasks to Todos | Douglas Barbosa Alexandre | 2016-02-20 | 1 | -3/+3 |
| | |||||
* | Rename IssuableBaseService#have_changes? to has_changes? | Douglas Barbosa Alexandre | 2016-02-20 | 1 | -1/+1 |
| | |||||
* | Ensure that we only have one task per issue/mr | Douglas Barbosa Alexandre | 2016-02-20 | 1 | -0/+5 |
| | |||||
* | Create a pending task when a user is mentioned when edit a issue/mr/note | Douglas Barbosa Alexandre | 2016-02-20 | 1 | -1/+1 |
| | |||||
* | Rename TaskService#mark_as_done to mark_pending_tasks_as_done | Douglas Barbosa Alexandre | 2016-02-20 | 1 | -1/+1 |
| | |||||
* | Move common behavior to to IssuableBaseService | Douglas Barbosa Alexandre | 2016-02-20 | 1 | -17/+0 |
| | |||||
* | Marks pending tasks for an user as done when he edit a MR | Douglas Barbosa Alexandre | 2016-02-20 | 1 | -0/+21 |
| | |||||
* | Create a pending task when an MR is assigned to someone | Douglas Barbosa Alexandre | 2016-02-20 | 1 | -0/+1 |
| | |||||
* | Marks pending tasks for an user as done when he edit an issue | Douglas Barbosa Alexandre | 2016-02-20 | 1 | -1/+1 |
| | |||||
* | Generate system note after Task item has been updated on Issue or Merge ↵ | Ruben Davila | 2015-11-19 | 1 | -4/+0 |
| | | | | | | | | | | | | 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. | ||||
* | Refactor similar code for Issue and MR update service | Dmitriy Zaporozhets | 2015-11-17 | 1 | -30/+9 |
| | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | ||||
* | Split huge method MergeRequests::UpdateService#execute | Dmitriy Zaporozhets | 2015-11-13 | 1 | -22/+25 |
| | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | ||||
* | Use issue editor as cross reference comment author when issue is edited with ↵edit-new-cross-references | Douwe Maan | 2015-10-28 | 1 | -1/+1 |
| | | | | a new mention. | ||||
* | Simplify code around (cross)-references | Douwe Maan | 2015-10-12 | 1 | -1/+1 |
| | |||||
* | Show who last edited a comment if it wasn't the original author | Douwe Maan | 2015-07-30 | 1 | -1/+1 |
| | |||||
* | Rename `notice_added_references` to `create_new_cross_references!`rs-issue-1773 | Robert Speicher | 2015-07-13 | 1 | -1/+1 |
| | |||||
* | Simplify Issue and MergeRequest UpdateServicesrs-remove-task-states | Robert Speicher | 2015-07-08 | 1 | -6/+2 |
| | | | | Now that those pesky task states are gone, these become a lot simpler. | ||||
* | Remove unused `task_[un]check` states from Update services | Robert Speicher | 2015-07-08 | 1 | -5/+1 |
| | | | | These were left over from the old Task handling. | ||||
* | Dont allow set assignee, milestone or labels if user is guest | Dmitriy Zaporozhets | 2015-06-25 | 1 | -0/+1 |
| | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | ||||
* | Disable changing of the source branch in merge request update API | Stan Hu | 2015-05-29 | 1 | -1/+2 |
| | |||||
* | Support editing target branch of merge request | Stan Hu | 2015-05-29 | 1 | -1/+12 |
| | | | | | Closes https://github.com/gitlabhq/gitlabhq/issues/7105 See: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/130 | ||||
* | Add a note when an Issue or Merge Request's title changes | Robert Speicher | 2015-05-26 | 1 | -0/+4 |
| | |||||
* | No magic numbers for issues filtering | Dmitriy Zaporozhets | 2015-03-26 | 1 | -2/+2 |
| | |||||
* | Make sure issue assignee is properly reset. | Douwe Maan | 2015-03-24 | 1 | -0/+3 |
| | |||||
* | Add labels notes | Nikita Verkhovin | 2015-02-08 | 1 | -0/+10 |
| | |||||
* | Organize event order execution when update issue or mr | Dmitriy Zaporozhets | 2015-01-25 | 1 | -1/+1 |
| | |||||
* | Add action property to merge request hook | jubianchi | 2015-01-20 | 1 | -1/+1 |
| |