Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Enable the Layout/ExtraSpacing cop56392-enable-the-layout-extraspacing-cop | Rémy Coutable | 2019-01-24 | 1 | -1/+1 |
| | | | | Signed-off-by: Rémy Coutable <remy@rymai.me> | ||||
* | Remove caching of CSV file49231-import-issues-csv | Heinrich Lee Yu | 2019-01-07 | 1 | -1/+1 |
| | | | | Load whole file in memory to simplify code | ||||
* | Improve email messages | Heinrich Lee Yu | 2019-01-07 | 1 | -7/+7 |
| | | | | Also refactored cleanup view to use the same localized string | ||||
* | Import CSV Backend | Heinrich Lee Yu | 2019-01-07 | 1 | -0/+64 |
| | | | | Process CSV uploads async using a worker then email results | ||||
* | Delete confidential issue todos for guests | Felipe Artur | 2018-12-14 | 1 | -1/+1 |
| | | | | | Fix leaking information of confidential issues on TODOs when user is downgraded to guest access. | ||||
* | Extract code from Issues::MoveServicece-3777-promote-to-epic | Jarka Košanová | 2018-11-12 | 1 | -256/+12 |
| | | | | | | | Extract code to make it easier reusable - introduce AttributesRewriter and ContentRewriter - support group entites when rewriting content - make Uploader copy_to working for Namespaces | ||||
* | Add email for milestone change | Chantal Rollison | 2018-11-02 | 1 | -2/+46 |
| | |||||
* | Move issue related_branches to service | Felipe Artur | 2018-10-04 | 1 | -0/+39 |
| | | | | | Moves the related_branches method from Issue model to RelatedBranchesService | ||||
* | Use ResourceLabelEvent for tracking label changes | Jan Provaznik | 2018-09-07 | 2 | -4/+16 |
| | |||||
* | Only load issue notes once when getting related MRs43096-controller-projects-issuescontroller-referenced_merge_requests-json-executes-more-than-100-sql-queries | Sean McGivern | 2018-08-21 | 1 | -0/+6 |
| | | | | | | | As we always call both methods from the controller - and elsewhere we call the more general method - and one uses all notes and the other uses system notes, then we should just load the notes and their authors once, and filter on the Ruby side. | ||||
* | Fix CI pipelines N+1 in Issues::ReferencedMergeRequestsService | Sean McGivern | 2018-08-21 | 1 | -0/+18 |
| | | | | | | | | | Whether the preloading belongs in the service or the controller is arguable, here. As the service is only used for one controller action, it seems reasonable to put it in the service, but that is not a definitive answer. Adding the preloads for MR project routes here doesn't seem to work, perhaps because of https://github.com/rails/rails/issues/32140. | ||||
* | Fix routes N+1 in Issues::ReferencedMergeRequestsService#execute | Sean McGivern | 2018-08-21 | 1 | -0/+13 |
| | | | | | Sorting here needs the project routes to be loaded, including the namespace routes. | ||||
* | Fix authors N+1 in Issues::ReferencedMergeRequestsService | Sean McGivern | 2018-08-21 | 1 | -1/+25 |
| | | | | | | | | `#referenced_merge_requests` preloaded too many associations. Award emoji, for instance, are completely unnecessary here. `#closed_by_merge_requests` had the opposite problem: `#all_references` needs each item's author, and these weren't preloaded. | ||||
* | Move Issue#{referenced,closed_by}_merge_requests to service | Sean McGivern | 2018-08-21 | 2 | -35/+72 |
| | | | | | | | | These methods don't really need to be on the Issue model. Issue#related_branches can also be moved to a service, but we can do that in a separate commit. This commit does not change any behaviour; it just moves code around, renames the service, and refactors the specs. | ||||
* | Merge branch 'issue_44821' into 'master' | Sean McGivern | 2018-08-06 | 1 | -1/+1 |
|\ | | | | | | | | | | | | | Retrieve merge request closing issues from database cache Closes #44821 See merge request gitlab-org/gitlab-ce!20911 | ||||
| * | Retrieve merge request closing issues from database cache | Felipe Artur | 2018-08-06 | 1 | -1/+1 |
| | | |||||
* | | Delete todos when users loses target read permissions | Jarka Kadlecová | 2018-07-30 | 1 | -0/+17 |
|/ | |||||
* | Resolve "Rename the `Master` role to `Maintainer`" Backend | Mark Chao | 2018-07-11 | 4 | -12/+12 |
| | |||||
* | Updates from `rubocop -a` | Lin Jen-Shin | 2018-07-09 | 1 | -2/+2 |
| | |||||
* | Keep label on issue when moving | Chantal Rollison | 2018-06-27 | 1 | -3/+23 |
| | |||||
* | Add a cop for `FinderMethods`bvl-finder-methods-cop | Bob Van Landuyt | 2018-06-20 | 1 | -3/+1 |
| | | | | | | | This notifies developers when calling `find(_by!)` chained on `execute`. And suggests using the methods from `FinderMethods`. These will perform the correct authorization checks on the resource when it is found. | ||||
* | Resolve "Update `updated_at` on an issue/mr on every issue/mr changes" | 🙈 jacopo beschi 🙉 | 2018-06-01 | 1 | -1/+7 |
| | |||||
* | Improve specs and docs | Felipe Artur | 2018-04-05 | 1 | -2/+4 |
| | |||||
* | Merge branch 'issue_44551' into 'master' | Sean McGivern | 2018-04-05 | 1 | -0/+31 |
|\ | | | | | | | | | | | | | Fix 404 in group boards when moving issue between lists Closes #44551 See merge request gitlab-org/gitlab-ce!18064 | ||||
| * | Fix 404 in group boards when moving issue between listsissue_44551 | Felipe Artur | 2018-04-03 | 1 | -0/+31 |
| | | |||||
* | | Merge branch '41967_issue_api_closed_by_info' into 'master' | Sean McGivern | 2018-03-30 | 1 | -0/+4 |
|\ \ | |/ |/| | | | | | Add closed by information to issue API See merge request gitlab-org/gitlab-ce!17042 | ||||
| * | added forign key and specs | haseeb | 2018-02-28 | 1 | -0/+4 |
| | | |||||
* | | Merge remote-tracking branch 'origin/master' into ↵object-storage-ee-to-ce-backport | Kamil Trzciński | 2018-02-28 | 2 | -0/+57 |
|\ \ | |/ | | | | | object-storage-ee-to-ce-backport | ||||
| * | Remember assignee when moving an issue41949-move | Jan Provaznik | 2018-02-18 | 1 | -0/+22 |
| | | | | | | | | Related to #41949 | ||||
| * | Merge branch '26466-natural-sort-mrs' into 'master' | Sean McGivern | 2018-02-13 | 1 | -0/+35 |
| |\ | | | | | | | | | | | | | | | | | | | Group MR numbers on issue show page by project and namespace Closes #26466 See merge request gitlab-org/gitlab-ce!8494 | ||||
| | * | Change issue show page to group MRs by projects and namespaces | Jeff Stubler | 2018-02-13 | 1 | -0/+35 |
| | | | | | | | | | | | | Closes #26466. | ||||
* | | | Merge commit '8af23def1d6' into object-storage-ee-to-ce-backport | Kamil Trzciński | 2018-02-28 | 3 | -6/+15 |
|\ \ \ | |||||
| * | | | Revert "Merge branch '3867-port-to-ce' into 'master'" | Kamil Trzciński | 2018-02-28 | 1 | -2/+2 |
| |/ / | | | | | | | | | | | | | This reverts commit 54a575f1bbba44573ab92dc58a4242f1ee734c5d, reversing changes made to c63af942e5baf7849a94fa99da8494bcba28e3f8. | ||||
| * | | port of 594e6a0a625^..f74c90f68c6 | Micaël Bergeron | 2018-02-01 | 1 | -2/+2 |
| | | | |||||
| * | | Make user/author use project.creator in most factories | Rémy Coutable | 2018-01-31 | 2 | -2/+3 |
| |/ | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me> | ||||
| * | Execute system hooks after-commit when executing project hooksdm-project-system-hooks-in-transaction | Douwe Maan | 2018-01-24 | 1 | -1/+3 |
| | | |||||
| * | Fix bug where award emojis would be lost when moving issues between projectssh-fix-award-emoji-move-issues | Stan Hu | 2018-01-10 | 1 | -3/+9 |
| | | | | | | | | Closes #33423 | ||||
* | | Merge branch '4163-move-uploads-to-object-storage' into 'master' | Sean McGivern | 2018-02-28 | 1 | -1/+1 |
|/ | | | | | | | | Move uploads to object storage Closes #4163 See merge request gitlab-org/gitlab-ee!3867 | ||||
* | Merge branch '36782-replace-team-user-role-with-add_role-user-in-specs' into ↵ | Rémy Coutable | 2017-12-23 | 7 | -30/+30 |
|\ | | | | | | | | | | | | | | | | | 'master' Resolve "Replace '.team << [user, role]' with 'add_role(user)' in specs" Closes #36782 See merge request gitlab-org/gitlab-ce!16069 | ||||
| * | Replace '.team << [user, role]' with 'add_role(user)' in specs36782-replace-team-user-role-with-add_role-user-in-specs | blackst0ne | 2017-12-22 | 7 | -30/+30 |
| | | |||||
* | | Execute project hooks and services after commit when moving an issuedm-issue-move-transaction-error | Douwe Maan | 2017-12-22 | 1 | -0/+12 |
|/ | |||||
* | Don't create todos for old issue assignees | Jarka Kadlecova | 2017-10-09 | 1 | -0/+24 |
| | |||||
* | Merge remote-tracking branch 'origin/master' into 18608-lock-issues | Luke "Jared" Bennett | 2017-09-20 | 3 | -2/+9 |
|\ | |||||
| * | Fix refreshing of issues/MR count cachesfix-counts-caching | Yorick Peterse | 2017-09-19 | 3 | -2/+9 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This ensures the open issues/MR count caches are refreshed properly when creating new issues or MRs. This MR also includes a change to the cache keys to ensure all caches are rebuilt on the fly. This particular problem was not caught in the test suite due to a null cache being used, resulting in all calls that would use a cache using the underlying data directly. In production the code would fail because a newly saved record returns an empty hash in #changes meaning checks such as `state_changed? || confidential_changed?` would return false for new rows, thus never updating the counters. Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/38061 | ||||
* | | Create system notes for MR too, improve doc + clean up code | Jarka Kadlecova | 2017-09-14 | 1 | -0/+7 |
| | | |||||
* | | Support discussion locking in the backend | Jarka Kadlecova | 2017-09-14 | 1 | -1/+4 |
|/ | |||||
* | Merge branch 'master' of gitlab.com:gitlab-org/gitlab-ce into ↵ | Simon Knox | 2017-09-06 | 2 | -2/+22 |
|\ | | | | | | | ee_issue_928_backport | ||||
| * | Fix a spec failure dependent on load orderrs-fix-master | Robert Speicher | 2017-09-01 | 1 | -1/+1 |
| | | | | | | | | | | | | We were incorrectly stubbing `Issues::MoveService#execute`. This only caused a failure when another spec caused it to be autoloaded -- `spec/controllers/projects/issues_controller_spec.rb` in this case. | ||||
| * | Merge branch 'move-action' into 'master' | Sean McGivern | 2017-09-01 | 1 | -0/+20 |
| |\ | | | | | | | | | | | | | Merge request to address /move functionality. See merge request !13436 | ||||
| | * | /move project functionality. Squash commit. | Manolis Mavrofidis | 2017-08-31 | 1 | -0/+20 |
| | | | | | | | | | | | | | | | Update to add all issues to be updated. Added functionality, proper tests, documentation and changelog. |