Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | 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 | ||||
| * | Backport backend work for time tracking. | Ruben Davila | 2017-01-15 | 1 | -0/+12 |
| | | |||||
* | | support `/merge` slash comand for MRs | Jarka Kadlecova | 2017-01-11 | 1 | -0/+11 |
|/ | |||||
* | Check if user can read issue before being assignedissue_22664 | Felipe Artur | 2016-12-27 | 1 | -0/+2 |
| | |||||
* | Does not raise error when Note not found when processing NewNoteWorker | Oswaldo Ferreira | 2016-11-11 | 1 | -4/+33 |
| | | | | - Also remove unnecessary param | ||||
* | Ensure we have a user before checking for their permission in ↵ | Rémy Coutable | 2016-09-20 | 1 | -0/+69 |
| | | | | | | Notes::SlashCommandsService Signed-off-by: Rémy Coutable <remy@rymai.me> | ||||
* | Remove unneeded aliases | Douwe Maan | 2016-08-18 | 1 | -4/+4 |
| | |||||
* | Fixed specs and fixes based on failing specs | Douwe Maan | 2016-08-16 | 2 | -12/+23 |
| | |||||
* | New Notes::SlashCommandsService service | Rémy Coutable | 2016-08-13 | 2 | -9/+143 |
| | | | | | | Check for update_issuable permission in Notes::SlashCommandsService Signed-off-by: Rémy Coutable <remy@rymai.me> | ||||
* | Support slash commands in noteable description and notes | Rémy Coutable | 2016-08-13 | 1 | -7/+16 |
| | | | | | | | | | | | | | | | | | | | | | | | | | 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> | ||||
* | Avoid `describe`-ing symbols in specsrs-describe-symbols | Robert Speicher | 2016-07-12 | 2 | -2/+2 |
| |