Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Move some quick actions feature specs to unit tests | Felipe Artur | 2019-06-18 | 1 | -49/+98 |
| | | | | | Move some feature specs for issues/merge requests quick actions to unit tests. They are taking too long to run on the pipelines. | ||||
* | Add frozen_string_literal to spec/servicesfrozen_string_literal_spec_services | Thong Kuah | 2019-04-12 | 8 | -0/+16 |
| | | | | Probably useful as we often move these files to "new" files. | ||||
* | Merge branch 'port-6539-extract-ee-specific-code' into 'master' | Robert Speicher | 2019-04-05 | 1 | -0/+1 |
|\ | | | | | | | | | Extract EE specific files/lines for quick actions tests See merge request gitlab-org/gitlab-ce!26872 | ||||
| * | Extract EE specific files/lines for quick actions testsport-6539-extract-ee-specific-code | Alexandru Croitor | 2019-04-03 | 1 | -0/+1 |
| | | |||||
* | | Remove reply_to_individual_notes feature flag58644-remove-reply_to_individual_notes-feature-flag | Heinrich Lee Yu | 2019-04-03 | 2 | -58/+19 |
|/ | | | | Also removes specs related to the flag | ||||
* | Show commands applied message when promoting issues | Felipe Artur | 2019-03-04 | 2 | -12/+25 |
| | | | | | Fix 'commands applied' messages not being shown when issue is promoted to epic using slash commands. | ||||
* | Fix polling for transformed individual notes30299-fix-polling-for-transformed-notes | Heinrich Lee Yu | 2019-02-13 | 1 | -1/+8 |
| | | | | | Sets `updated_at` so that the transformed note is returned together with the new reply | ||||
* | Add reply to notes to turn into discussions | Heinrich Lee Yu | 2019-02-06 | 2 | -0/+77 |
| | |||||
* | Prevent comments by email when issue is locked | Heinrich Lee Yu | 2019-01-31 | 2 | -0/+13 |
| | | | | | | | | | This changes the permission check so it uses the policy on Noteable instead of Project. This prevents bypassing of rules defined in Noteable for locked discussions and confidential issues. Also rechecks permissions when reply_to_discussion_id is provided since the discussion_id may be from a different noteable. | ||||
* | Allow suggesting single line changes in diffs | Oswaldo Ferreira | 2018-12-13 | 1 | -0/+23 |
| | |||||
* | Comment on any expanded diff line on MRsosw-comment-on-any-line-on-diffs-w-feature-flag | Oswaldo Ferreira | 2018-11-09 | 2 | -0/+84 |
| | |||||
* | Revert "Merge branch 'osw-comment-on-any-line-on-diffs' into 'master'"osw-revert-comment-in-any-diff-line | Oswaldo Ferreira | 2018-11-07 | 2 | -84/+0 |
| | | | | | This reverts commit a82a595728d54bdc12e51dfcfb22e9eddc449143, reversing changes made to e7df959b8f99875edd246c7ac7779c3203e8755e. | ||||
* | Merge branch 'osw-comment-on-any-line-on-diffs' into 'master' | Douwe Maan | 2018-11-07 | 2 | -0/+84 |
|\ | | | | | | | | | Comment on any expanded diff line on MRs See merge request gitlab-org/gitlab-ce!22398 | ||||
| * | Comment on any expanded diff line on MRs | Oswaldo Ferreira | 2018-11-06 | 2 | -0/+84 |
| | | |||||
* | | Fix spec in EE which now trigger access to repository | Mark Chao | 2018-11-07 | 1 | -1/+1 |
|/ | |||||
* | Auto resolve new notes of resolved discussions | Paul Slaughter | 2018-09-24 | 1 | -0/+15 |
| | | | | | | | | | | | | | | **Why?** The previous behavior had resolved discussions being unresolved when commented on. This was strange UX, especially since there is a separate button for "Comment & unresolve discussion". https://gitlab.com/gitlab-org/gitlab-ce/issues/24128 **Note:** - Also adds changelog | ||||
* | Rename Commits::UpdateService to Commits::TagService | Peter Leitzen | 2018-08-10 | 1 | -2/+2 |
| | |||||
* | Commit notes now support quick actions | Peter Leitzen | 2018-08-10 | 1 | -42/+4 |
| | |||||
* | Convert use of any_instance as it does not work with prepend | Mark Chao | 2018-08-07 | 1 | -1/+3 |
| | |||||
* | Resolve "Rename the `Master` role to `Maintainer`" Backend | Mark Chao | 2018-07-11 | 4 | -13/+13 |
| | |||||
* | Persist truncated note diffs on a new table45190-create-notes-diff-files | Oswaldo Ferreira | 2018-05-24 | 1 | -0/+82 |
| | | | | | We request Gitaly in a N+1 manner to build discussion diffs. Once the diffs are from different revisions, it's hard to make a single request to the service in order to build the whole response. With this change we solve this problem and simplify a lot fetching this piece of info. | ||||
* | Merge request and commit discussions API | Jan Provaznik | 2018-05-01 | 1 | -0/+23 |
| | |||||
* | Support Markdown rendering using multiple projectsrendering-markdown-multiple-projects | Yorick Peterse | 2018-04-11 | 1 | -10/+15 |
| | | | | | | | | | | | | | | | | | | | This refactors the Markdown pipeline so it supports the rendering of multiple documents that may belong to different projects. An example of where this happens is when displaying the event feed of a group. In this case we retrieve events for all projects in the group. Previously we would group events per project and render these chunks separately, but this would result in many SQL queries being executed. By extending the Markdown pipeline to support this out of the box we can drastically reduce the number of SQL queries. To achieve this we introduce a new object to the pipeline: Banzai::RenderContext. This object simply wraps two other objects: an optional Project instance, and an optional User instance. On its own this wouldn't be very helpful, but a RenderContext can also be used to associate HTML documents with specific Project instances. This work is done in Banzai::ObjectRenderer and allows us to reuse as many queries (and results) as possible. | ||||
* | Merge branch 'jej/mattermost-notification-confidentiality-10-6' into ↵ | Douwe Maan | 2018-04-05 | 1 | -0/+18 |
| | | | | | | | | | | 'security-10-6' [10.6] Prevent notes on confidential issues from being sent to chat See merge request gitlab/gitlabhq!2366 # Conflicts: # app/helpers/services_helper.rb | ||||
* | Fix quick actions for users who cannot update issues and MRs43334-reply-by-email-did-not-pick-up-unsubscribe-quick-action | Sean McGivern | 2018-03-02 | 2 | -36/+45 |
| | | | | | | There are several quick actions now that don't need this access - /todo and /unsubscribe for instance - but when these were first added, there weren't. Quick actions are now responsible for checking their own permissions. | ||||
* | Replace '.team << [user, role]' with 'add_role(user)' in specs36782-replace-team-user-role-with-add_role-user-in-specs | blackst0ne | 2017-12-22 | 4 | -10/+10 |
| | |||||
* | Reset todo counters when the target is deleted40871-todo-notification-count-shows-notification-without-having-a-todo | Sean McGivern | 2017-12-18 | 1 | -3/+13 |
| | | | | | | When the target is deleted, todos are destroyed, but we did not reset the todo cache for users with todos on the deleted target. This would only update after the next time the todo cache was updated for that user. | ||||
* | Resolve "DashboardController#activity.json is slow due to SQL" | Francisco Javier López | 2017-11-06 | 1 | -0/+31 |
| | |||||
* | Change all `:empty_project` to `:project`rs-empty_project-default | Robert Speicher | 2017-08-02 | 5 | -6/+6 |
| | |||||
* | Use described_class when possible | Rémy Coutable | 2017-07-27 | 1 | -1/+1 |
| | | | | Signed-off-by: Rémy Coutable <remy@rymai.me> | ||||
* | Remove superfluous lib: true, type: redis, service: true, models: true, ↵ | Rémy Coutable | 2017-07-27 | 6 | -6/+6 |
| | | | | | | services: true, no_db: true, api: true Signed-off-by: Rémy Coutable <remy@rymai.me> | ||||
* | Rename "Slash commands" to "Quick actions" | Eric Eastwood | 2017-06-15 | 1 | -6/+6 |
| | | | | | | | | | | | | | 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` | ||||
* | Correct RSpec/SingleLineHook cop offenses | Robert Speicher | 2017-06-14 | 1 | -1/+3 |
| | |||||
* | Update diff discussion position per discussion instead of per notedm-update-discussion-diff-position | Douwe Maan | 2017-05-31 | 1 | -193/+0 |
| | |||||
* | Add system note with link to diff comparison when MR discussion becomes outdated | Douwe Maan | 2017-05-23 | 1 | -1/+19 |
| | |||||
* | Merge branch 'mia_backort' into 'master' | Jacob Schatz | 2017-05-05 | 1 | -2/+29 |
|\ | | | | | | | | | Backport of Multiple Assignees feature See merge request !11089 | ||||
| * | Address static analyzer warningmia_backort | Valery Sizov | 2017-05-05 | 1 | -1/+1 |
| | | |||||
| * | Multiple issue assignee: CE restriction for multiple assignees | Valery Sizov | 2017-05-05 | 1 | -0/+27 |
| | | |||||
| * | Backport of multiple_assignees_feature [ci skip] | Valery Sizov | 2017-05-04 | 1 | -2/+2 |
| | | |||||
* | | Support comments for personal snippets | Jarka Kadlecova | 2017-05-05 | 1 | -1/+73 |
|/ | |||||
* | Don't use original_discussion_id | Douwe Maan | 2017-04-05 | 1 | -1/+1 |
| | |||||
* | Add specs | Douwe Maan | 2017-04-05 | 2 | -5/+37 |
| | |||||
* | Add option to start a new discussion on an MR | Douwe Maan | 2017-04-05 | 2 | -0/+8 |
| | |||||
* | Create todos only for new mentions | Jarka Kadlecova | 2017-03-29 | 1 | -3/+13 |
| | |||||
* | Use `:empty_project` where possible in service specs | Robert Speicher | 2017-03-27 | 1 | -1/+1 |
| | |||||
* | add /award slash command | mhasbini | 2017-02-28 | 1 | -39/+11 |
| | | | | add /award slash command; Allow posting of just an emoji in comment | ||||
* | Fix inconsistent naming for services that delete things | dixpac | 2017-02-08 | 1 | -1/+1 |
| | | | | | | * Changed name of delete_user_service and worker to destroy * Move and change delete_group_service to Groups::DestroyService * Rename Notes::DeleteService to Notes::DestroyService | ||||
* | address comments | Jarka Kadlecova | 2017-01-25 | 1 | -9/+9 |
| | |||||
* | Support notes without project | Jarka Kadlecova | 2017-01-18 | 1 | -4/+31 |
| | |||||
* | Merge branch 'backport-time-tracking-ce' into 'master' | Douwe Maan | 2017-01-18 | 1 | -0/+12 |
|\ | | | | | | | | | Backport timetracking to CE See merge request !8195 |