summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Fix for Emojinotes_fix_stablenotes_fixValery Sizov2015-11-204-4/+6
|
* Merge branch 'update-lfs-docs' into 'master' Marin Jankovski2015-11-201-3/+2
|\ | | | | | | | | | | | | Update LFS docs for cloning Cloning the repo already downloads the LFS tracked files. See merge request !1839
| * Update LFS docs for cloning [ci skip]update-lfs-docsRobert Schilling2015-11-191-3/+2
| |
* | Merge branch 'rs-mr-reference-copy' into 'master' Robert Speicher2015-11-201-2/+4
|\ \ | | | | | | | | | | | | Add clipboard button to merge request cross-project reference See merge request !1847
| * | Add clipboard button to merge request cross-project referenceRobert Speicher2015-11-191-2/+4
|/ /
* | Merge branch 'rs-safari-clipboard-fallback' into 'master' Robert Speicher2015-11-191-5/+16
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | Add a fallback for Safari copy-to-clipboard Also, hide the tooltip in a less stupid way. Closes #3547 See merge request !1844
| * | Add a fallback for Safari copy-to-clipboardrs-safari-clipboard-fallbackRobert Speicher2015-11-191-5/+16
| | | | | | | | | | | | | | | | | | Also, hide the tooltip in a less stupid way. Closes #3547
* | | Merge branch 'ci-yaml-validation' into 'master' Kamil Trzciński2015-11-196-22/+33
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commits without .gitlab-ci.yml are marked as skipped - Commits without .gitlab-ci.yml are marked as skipped - Save detailed error when YAML syntax This also fixes: #3521 #3546 /cc @jacobvosmaer See merge request !1827
| * | | Don't rescue Exception, but StandardErrorci-yaml-validationKamil Trzcinski2015-11-192-8/+5
| | | |
| * | | Fix testsKamil Trzcinski2015-11-191-2/+2
| | | |
| * | | Since GitLab CI is enabled by default, remove enabling it by pushing ↵Kamil Trzcinski2015-11-192-12/+1
| | | | | | | | | | | | | | | | .gitlab-ci.yml
| * | | Fix testsKamil Trzcinski2015-11-192-20/+20
| | | |
| * | | Commits without .gitlab-ci.yml are marked as skippedKamil Trzcinski2015-11-193-3/+28
| | | | | | | | | | | | | | | | - Save detailed error when YAML syntax
* | | | Merge branch 'emoji_votes' into 'master' Dmitriy Zaporozhets2015-11-1933-438/+431
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Award Emoji This it first iteration of award emoji feature. We have plan to extend emoji picker by the next release. For now, you can add award by clicking to the emoji picker or posting a regular comment with emoji like ":+1:" and any other. You can post not only emoji that listed in the emoji picker. See merge request !1825
| * | | | fox testsemoji_votesValery Sizov2015-11-192-28/+27
| | | | |
| * | | | adressing commentsValery Sizov2015-11-195-20/+21
| | | | |
| * | | | added specsValery Sizov2015-11-193-0/+73
| | | | |
| * | | | added spinach testsValery Sizov2015-11-197-96/+83
| | | | |
| * | | | Few minor improvements to emoji awards UIDmitriy Zaporozhets2015-11-191-3/+2
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | | Improve UI for emoji awardsDmitriy Zaporozhets2015-11-193-21/+36
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | | satisfy rubocopValery Sizov2015-11-193-8/+8
| | | | |
| * | | | fix schemaValery Sizov2015-11-191-1/+1
| | | | |
| * | | | add changelogValery Sizov2015-11-191-0/+1
| | | | |
| * | | | addressing commentsValery Sizov2015-11-195-9/+15
| | | | |
| * | | | improve styleValery Sizov2015-11-192-2/+5
| | | | |
| * | | | better regexpValery Sizov2015-11-191-1/+1
| | | | |
| * | | | Emoji: fix image of emoji when it is submitted via commentValery Sizov2015-11-196-9/+17
| | | | |
| * | | | fixValery Sizov2015-11-191-10/+9
| | | | |
| * | | | Emoji: refactoringValery Sizov2015-11-193-1/+11
| | | | |
| * | | | award for merge requests[ci skip]Valery Sizov2015-11-193-10/+6
| | | | |
| * | | | add stats on hoverValery Sizov2015-11-193-3/+41
| | | | |
| * | | | replace emoji references from class name to data [ci skip]Valery Sizov2015-11-193-22/+26
| | | | |
| * | | | css improvementsValery Sizov2015-11-194-9/+19
| | | | |
| * | | | award emojiValery Sizov2015-11-1924-361/+203
| | | | |
* | | | | Merge branch 'missing_dot_confirm_modal' into 'master' Dmitriy Zaporozhets2015-11-191-1/+1
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | Add missing dot in confirm modal See merge request !1645
| * | | | Add missing dot in confirm modalAchilleas Pipinellis2015-10-191-1/+1
| | | | |
* | | | | Merge branch 'finding-issues-by-labels-performance' into 'master' Dmitriy Zaporozhets2015-11-198-11/+87
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Improve performance of finding issues with/without labels The changes in this MR ultimately lead to finding issues with(out) labels being about 2x faster due to: 1. Newly added indexes on `issues.state` and `projects.visibility_level` 2. Adjusting the query so that finding issues for multiple projects is more efficient See merge request !1787
| * | | | | Align hash literals in IssuesFinder specfinding-issues-by-labels-performanceYorick Peterse2015-11-191-3/+3
| | | | | |
| * | | | | Changelog entry for finding issues performanceYorick Peterse2015-11-191-0/+1
| | | | | |
| * | | | | Use a JOIN in IssuableFinder#by_projectYorick Peterse2015-11-193-3/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When using IssuableFinder/IssuesFinder to find issues for multiple projects it's more efficient to use a JOIN + a "WHERE project_id IN" condition opposed to running a sub-query. This change means that when finding issues without labels we're now using the following SQL: SELECT issues.* FROM issues JOIN projects ON projects.id = issues.project_id LEFT JOIN label_links ON label_links.target_type = 'Issue' AND label_links.target_id = issues.id WHERE ( projects.id IN (...) OR projects.visibility_level IN (20, 10) ) AND issues.state IN ('opened','reopened') AND label_links.id IS NULL ORDER BY issues.id DESC; instead of: SELECT issues.* FROM issues LEFT JOIN label_links ON label_links.target_type = 'Issue' AND label_links.target_id = issues.id WHERE issues.project_id IN ( SELECT id FROM projects WHERE id IN (...) OR visibility_level IN (20,10) ) AND issues.state IN ('opened','reopened') AND label_links.id IS NULL ORDER BY issues.id DESC; The big benefit here is that in the last case PostgreSQL can't properly use all available indexes. In particular it ends up performing a sequence scan on the "label_links" table (processing around 290 000 rows). The new query is roughly 2x as fast as the old query.
| * | | | | Memoize IssuableFinder#projectsYorick Peterse2015-11-191-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since this method's returned data doesn't change between calls on the same IssuableFinder instance we can just memoize this similar to the "project" method.
| * | | | | Removed trailing whitespace from IssuableFinderYorick Peterse2015-11-191-2/+2
| | | | | |
| * | | | | Added benchmark for IssuesFinderYorick Peterse2015-11-191-0/+55
| | | | | |
| * | | | | Updated DB schema with new issues/projects indexesYorick Peterse2015-11-191-3/+3
| | | | | |
| * | | | | Added index on projects.visibility_levelYorick Peterse2015-11-191-0/+5
| | | | | |
| * | | | | Added index on issues.stateYorick Peterse2015-11-191-0/+5
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | This field is queried when filtering issues and due to the lack of an index would end up triggering a sequence scan.
* | | | | Merge branch 'dbalexandre/gitlab-ce-fix-personal-snippet-access-workflow' ↵Robert Speicher2015-11-195-23/+199
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | into 'master' Improve personal snippet access workflow. Replaces !1709 Fixes #3258 See merge request !1817
| * | | | | Rename `not_auth_*` ability methods to `anonymous_*`dbalexandre/gitlab-ce-fix-personal-snippet-access-workflowRobert Speicher2015-11-171-10/+9
| | | | | |
| * | | | | Fix changelogDouwe Maan2015-11-171-2/+1
| | | | | |
| * | | | | Minor refactoringDouwe Maan2015-11-171-8/+14
| | | | | |