Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Add latest changes from gitlab-org/gitlab@13-4-stable-ee | GitLab Bot | 2020-09-19 | 1 | -1/+3 |
| | |||||
* | Add latest changes from gitlab-org/gitlab@13-2-stable-ee | GitLab Bot | 2020-07-20 | 1 | -1/+1 |
| | |||||
* | Add latest changes from gitlab-org/gitlab@13-1-stable-ee | GitLab Bot | 2020-06-18 | 1 | -5/+25 |
| | |||||
* | Add latest changes from gitlab-org/gitlab@13-0-stable-ee | GitLab Bot | 2020-05-20 | 1 | -0/+18 |
| | |||||
* | Add latest changes from gitlab-org/gitlab@master | GitLab Bot | 2019-12-23 | 1 | -4/+4 |
| | |||||
* | Add latest changes from gitlab-org/gitlab@master | GitLab Bot | 2019-12-18 | 1 | -7/+113 |
| | |||||
* | Add latest changes from gitlab-org/gitlab@master | GitLab Bot | 2019-11-20 | 1 | -1/+3 |
| | |||||
* | Make diff_refs_match_commit validation reusable | Patrick Bajao | 2019-08-01 | 1 | -0/+4 |
| | | | | | Move it to DiffPositionableNote concern which will be re-used in EE in DraftNote model. | ||||
* | Avoid 500's when commit is not reachable | Oswaldo Ferreira | 2019-06-05 | 1 | -0/+8 |
| | |||||
* | Add # frozen_string_literal to spec/models | Thong Kuah | 2019-04-01 | 1 | -0/+2 |
| | | | | Adds `# frozen_string_literal: true` to spec/models ruby files | ||||
* | Implement multi-line suggestions filtering | Oswaldo Ferreira | 2019-03-20 | 1 | -0/+10 |
| | | | | | Implements the filtering logic for `suggestion:-x+y` syntax. | ||||
* | Add image_diff_note_on_merge_request factory | Douwe Maan | 2018-12-20 | 1 | -16/+1 |
| | |||||
* | Allow suggesting single line changes in diffs | Oswaldo Ferreira | 2018-12-13 | 1 | -0/+18 |
| | |||||
* | Persist truncated note diffs on a new table45190-create-notes-diff-files | Oswaldo Ferreira | 2018-05-24 | 1 | -1/+61 |
| | | | | | 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. | ||||
* | Use persisted diff data instead fetching Git on discussions | Oswaldo Ferreira | 2018-04-30 | 1 | -5/+28 |
| | | | | Today, when fetching diffs of a note, we always go to Gitaly in order to diff between commits and return the diff of each discussion note. With this change we avoid doing that for notes on the "current version" of the MR. | ||||
* | Improve performance of DiffDiscussion#truncated_diff_lines and ↵ | Douwe Maan | 2017-12-22 | 1 | -16/+0 |
| | | | | DiffNote#diff_line by removing expensive diff position calculation and comparison | ||||
* | adding view and feature specs | micael.bergeron | 2017-12-07 | 1 | -17/+12 |
| | |||||
* | Prevents position update for image diff notesissue_40058 | Felipe Artur | 2017-11-13 | 1 | -0/+6 |
| | |||||
* | Move line code generation into Gitlab::Gitconflict-resolution-refactor | Alejandro Rodríguez | 2017-10-12 | 1 | -1/+1 |
| | | | | Having a distinct class just for that was a bit overkill | ||||
* | Create a Gitlab::Git submodule for conlict-related files | Alejandro Rodríguez | 2017-10-12 | 1 | -1/+1 |
| | | | | Rename classes to (hopefully) clearer names while we're doing that. | ||||
* | Move Gitlab::Diff::LineCode to module Gitlab::Git | Alejandro Rodríguez | 2017-10-12 | 1 | -1/+1 |
| | |||||
* | Commenting on image diffs | Felipe Artur | 2017-10-07 | 1 | -3/+37 |
| | |||||
* | Remove superfluous lib: true, type: redis, service: true, models: true, ↵ | Rémy Coutable | 2017-07-27 | 1 | -1/+1 |
| | | | | | | services: true, no_db: true, api: true Signed-off-by: Rémy Coutable <remy@rymai.me> | ||||
* | Update diff discussion position per discussion instead of per notedm-update-discussion-diff-position | Douwe Maan | 2017-05-31 | 1 | -23/+4 |
| | |||||
* | Pass fallback_diff_refs to Diff::File instead of using view helpers | Douwe Maan | 2017-05-23 | 1 | -2/+2 |
| | |||||
* | Allow commenting on older versions of the diff and comparisons between diff ↵dm-link-discussion-to-outdated-diff | Douwe Maan | 2017-05-03 | 1 | -17/+35 |
| | | | | versions | ||||
* | Add specs | Douwe Maan | 2017-04-10 | 1 | -0/+17 |
| | |||||
* | Don't use original_discussion_id | Douwe Maan | 2017-04-05 | 1 | -25/+0 |
| | |||||
* | Add specs | Douwe Maan | 2017-04-05 | 1 | -1/+23 |
| | |||||
* | Add option to start a new discussion on an MR | Douwe Maan | 2017-04-05 | 1 | -283/+4 |
| | |||||
* | Use `:empty_project` where possible in model specsrs-empty_project-models | Robert Speicher | 2017-01-26 | 1 | -2/+2 |
| | |||||
* | Optimize discussion notes resolving and unresolving21109-discussion-resolve-runs-a-single-update-query-per-note-but-should-run-a-single-update-query-for-all-notes-instead | Alejandro Rodríguez | 2016-09-06 | 1 | -0/+37 |
| | | | | | | | Use `update_all` to only require one query per discussion to update the notes resolved status. Some changes had to be made to the discussion spec to accout for the fact that notes are not individually updated now | ||||
* | Store discussion_id on Note for faster discussion lookup. | Douwe Maan | 2016-08-17 | 1 | -0/+50 |
| | |||||
* | Address review feedback | Douwe Maan | 2016-08-16 | 1 | -1/+1 |
| | |||||
* | Add specs for new Note and DiffNote methods. | Douwe Maan | 2016-08-12 | 1 | -0/+246 |
| | |||||
* | Speedup DiffNote#active? on discussions, preloading noteables and avoid ↵ | Paco Guzman | 2016-08-02 | 1 | -2/+2 |
| | | | | | | | | | | | touching git repository to return diff_refs when possible - Preloading noteable we share the same noteable instance when more than one discussion refers to the same noteable. - Any other call to that object that is cached in that object will be for any discussion. - In those cases where merge_request_diff has all the sha stored to build a diff_refs get that diff_refs using directly those sha instead accessing to the git repository to first get the commits and later the sha. | ||||
* | Make `DiffNote#update_position` private | Douwe Maan | 2016-07-07 | 1 | -39/+44 |
| | |||||
* | Add tests for DiffNote | Douwe Maan | 2016-07-06 | 1 | -0/+186 |