summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Use just SQL to check is a user can admin_issue on a project17932-move-to-project-dropdown-sqlPaco Guzman2016-08-095-32/+111
| | | | | | | Using offset pagination instead pages to avoid a count query Tradeoff - we duplicate how we check admin_issue in a SQL relation in the Ability class
* user.can?(:admin_issue, project) improvementPaco Guzman2016-08-092-2/+15
| | | This make sense when checking a bunch of projects
* Move to project dropdown with infinite scroll for better performancePaco Guzman2016-08-087-116/+296
|
* Merge branch 'fix/import-pending-user' into 'master' Rémy Coutable2016-08-083-5/+29
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Support pending invitation project members importing projects Adds support for pending invitation project members on Import/Export - previously the import would fail. Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/19973 - [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry added - Tests - [x] Added for this feature/bug - [x] All builds are passing - [x] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides) - [x] Branch has no merge conflicts with `master` (if you do - rebase it please) - [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) See merge request !5683
| * Support pending invitation project members importing projectsJames Lopez2016-08-083-5/+29
| |
* | Merge branch 'faster-cache-keys' into 'master' Rémy Coutable2016-08-085-0/+36
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Optimize "cache_key" using a concern ## What does this MR do? This MR adds a concern (used by Issue and Note) that provides an optimized version of Rails' `cache_key` method. See 77c8520e2ecd70520757aed0fbdf434643b60234 for more details. ## Are there points in the code the reviewer needs to double check? No, though a spell check would be appreciated. ## Why was this MR needed? When loading a lot of data from Redis (e.g. an issue with lots of notes) quite a large amount of time is spent in generating cache keys. This is due to multiple reasons such as: * Rails trying to figure out if it should use `updated_at` or `updated_on` using somewhat inefficient code * Rails relying on pluralization logic to figure out how to generate a cache namespace using a model name * Rails calling a whole bunch of methods in general in the process of generating cache keys In short, Rails is trying to cater to every possible use case, at the cost of performance. ## What are the relevant issue numbers? https://gitlab.com/gitlab-org/gitlab-ce/issues/13651 is not directly related but I ran into this `cache_key` problem when looking into said issue. See merge request !5715
| * | Added concern for a faster "cache_key" methodfaster-cache-keysYorick Peterse2016-08-085-0/+36
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This concern provides an optimized/simplified version of the "cache_key" method. This method is about 9 times faster than the default "cache_key" method. The produced cache keys _are_ different from the previous ones but this is worth the performance improvement. To showcase this I set up a benchmark (using benchmark-ips) that compares FasterCacheKeys#cache_key with the regular cache_key. The output of this benchmark was: Calculating ------------------------------------- cache_key 4.825k i/100ms cache_key_fast 21.723k i/100ms ------------------------------------------------- cache_key 59.422k (± 7.2%) i/s - 299.150k cache_key_fast 543.243k (± 9.2%) i/s - 2.694M Comparison: cache_key_fast: 543243.4 i/s cache_key: 59422.0 i/s - 9.14x slower To see the impact on real code I applied these changes and benchmarked Issue#referenced_merge_requests. For an issue referencing 10 merge requests these changes shaved off between 40 and 60 milliseconds.
* | Merge branch 'update-templates' into 'master' Rémy Coutable2016-08-083-19/+19
|\ \ | | | | | | | | | | | | | | | | | | Update templates There was a copy pasta in the templates, therefor another MR with updates. See merge request !5714
| * | Update templatesZ.J. van de Weg2016-08-083-19/+19
| | |
* | | Merge branch 'master' of gitlab.com:gitlab-org/gitlab-ceDmitriy Zaporozhets2016-08-08100-636/+1275
|\ \ \
| * \ \ Merge branch 'branch-permissions' into 'master' Jacob Schatz2016-08-0818-229/+350
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Branch permissions layout CE part of https://gitlab.com/gitlab-org/gitlab-ee/issues/179 Related javascript code has been refactored and also layout has been updated. ![Screen_Shot_2016-08-04_at_3.23.19_PM](/uploads/a0f6165e255b3f93dcb80eaa3f3318e4/Screen_Shot_2016-08-04_at_3.23.19_PM.png) See merge request !5652
| | * | | Ensure we are looking for the right dropdown inside the form wrapperbranch-permissionsAlfredo Sumaran2016-08-051-2/+2
| | | | |
| | * | | Set for for labels and ID for dropdowns on create formAlfredo Sumaran2016-08-052-5/+6
| | | | |
| | * | | Fix .panel-title styleAlfredo Sumaran2016-08-053-2/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since .panel-heading is a bit different from bootstrap, using .panel-title for h3 looks different. So with this .panel-title will look good again inheriting custom properties our app sets for .panel-heading
| | * | | Refine selector for form submit buttonAlfredo Sumaran2016-08-051-1/+1
| | | | |
| | * | | Fix spelling. `braches` to `branches`Alfredo Sumaran2016-08-051-1/+1
| | | | |
| | * | | Cache onSelect callback before passing to each dropdown instanceAlfredo Sumaran2016-08-051-7/+9
| | | | |
| | * | | Wrap param with parens for consistencyAlfredo Sumaran2016-08-051-1/+1
| | | | |
| | * | | Prevent setting HTML directly from the JSAlfredo Sumaran2016-08-052-3/+4
| | | | |
| | * | | camelCase param nameAlfredo Sumaran2016-08-051-4/+4
| | | | |
| | * | | Wrap classes with IIFE and define it inside gl namespaceAlfredo Sumaran2016-08-055-121/+141
| | | | |
| | * | | Add url to Wildcard protected branches documentationAlfredo Sumaran2016-08-051-1/+2
| | | | |
| | * | | Create protected branch dropdown directlyAlfredo Sumaran2016-08-052-16/+5
| | | | | | | | | | | | | | | | | | | | Since there’s only one dropdown of this type we don’t need a class to create this dropdown masively.
| | * | | Move classes into its own fileAlfredo Sumaran2016-08-052-13/+12
| | | | |
| | * | | No need to pass event as a parameterAlfredo Sumaran2016-08-053-8/+6
| | | | | | | | | | | | | | | | | | | | We should stop passing events as a parameter since we cannot call the method programatically without faking or creating an event even when that event is not entirely required.
| | * | | Add custom css class to each dropdown to fix failing specAlfredo Sumaran2016-08-052-5/+5
| | | | |
| | * | | Select first value by defaultAlfredo Sumaran2016-08-051-2/+11
| | | | |
| | * | | Remove unnecesary classes to generate dropdownsAlfredo Sumaran2016-08-053-30/+6
| | | | | | | | | | | | | | | | | | | | Since there are only one dropdown of each type there are no need to create a class to initialize multiple elements with the same CSS class name.
| | * | | Fix class names and move each class to its own fileAlfredo Sumaran2016-08-0510-74/+79
| | | | |
| | * | | Refactor of Protected Branch Edit ListAlfredo Sumaran2016-08-054-10/+79
| | | | |
| | * | | Update layout and JS for create protected branch.Alfredo Sumaran2016-08-0514-215/+259
| | | | | | | | | | | | | | | | | | | | Also updates protect branch list
| * | | | Merge branch 'add-mr-migration-guidelines' into 'master' Achilleas Pipinellis2016-08-081-0/+4
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add migration-related tips to the "Merge Request Guidelines" doc ## Why was this MR needed? To avoid issues like #20606 cropping up again. See merge request !5700
| | * | | | Add migration-related tips to the "Merge Request Guidelines" docadd-mr-migration-guidelinesTimothy Andrew2016-08-081-0/+4
| | | |/ / | | |/| | | | | | | | | | | | [ci-skip]
| * | | | Merge branch '18256-secret-token-docs' into 'master' Rémy Coutable2016-08-083-1/+6
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Document that webhook secret token is sent in X-Gitlab-Token HTTP header ## What does this MR do? Note that the secret token is sent in the X-Gitlab-Token header on the webhook documentation page, as well as directly below the secret token field on the webhook settings form. ## Are there points in the code the reviewer needs to double check? No. ## Why was this MR needed? It took me a while to figure out how to verify the token in my hook endpoint. Issue #18256 is where I found how to do it. ## What are the relevant issue numbers? #18256 ## Screenshots (if relevant) ## Does this MR meet the acceptance criteria? - [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry added - [x] [Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md) - [ ] API support added - Tests - [ ] Added for this feature/bug - [x] All builds are passing - [x] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides) - [x] Branch has no merge conflicts with `master` (if you do - rebase it please) - [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) Closes #18256 See merge request !5664
| | * | | | Move CHANGELOG entry to middle of current releaseSteve Halasz2016-08-051-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Hope to avoid CHANGELOG conflicts. Credit myself for the MR.
| | * | | | Merge branch 'master' into 18256-secret-token-docsSteve Halasz2016-08-05100-421/+989
| | |\ \ \ \
| | * | | | | Document that webhook secret token is sent in X-Gitlab-Token HTTP headerSteve Halasz2016-08-033-1/+6
| | | | | | |
| * | | | | | Merge branch 'backport-user-select-fix' into 'master' Rémy Coutable2016-08-081-16/+22
| |\ \ \ \ \ \ | | |_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fix: respect data-attribute 'skip-users' in user_selects Backport https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/604 to CE, so that JS file is the same in both versions. Thanks @hwdegroot! No CHANGELOG needed because this code path isn't used anywhere in CE at the moment. See merge request !5685
| | * | | | | fix: respect data-attribute 'skip-users' in user_selectsbackport-user-select-fixRik de Groot2016-08-051-16/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Respect parameters of each individual select, instead of the global. * Update skipped users in approver search when removed from default approvers.
| * | | | | | Merge branch 'simplify_intro_features_doc' into 'master' Achilleas Pipinellis2016-08-0821-58/+55
| |\ \ \ \ \ \ | | |_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Simplify feature introduction note See merge request !5705
| | * | | | | Simplify feature introduction notesimplify_intro_features_docAchilleas Pipinellis2016-08-0821-58/+55
| |/ / / / / | | | | | | | | | | | | | | | | | | [ci skip]
| * | | | | Merge branch 'doc-location-guidelines' into 'master' Achilleas Pipinellis2016-08-061-5/+60
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add doc guidelines on documents naming and location ## What does this MR do? Add guidelines on the structure of the documentation. ## What are the relevant issue numbers? https://gitlab.com/gitlab-org/gitlab-ce/issues/3349 See merge request !5641
| | * | | | | Add doc guidelines on documents naming and locationdoc-location-guidelinesAchilleas Pipinellis2016-08-031-5/+60
| | | | | | | | | | | | | | | | | | | | | | | | | | | | [ci skip]
| * | | | | | Merge branch 'development-docs-from-gdk' into 'master' Achilleas Pipinellis2016-08-061-4/+26
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add 'run tests' development docs from GDK This adds some documentation that lived in https://gitlab.com/gitlab-org/gitlab-development-kit but which belongs more in here. See merge request !5684
| | * | | | | | Add 'run tests' docs from GDKJacob Vosmaer2016-08-051-4/+26
| | | |/ / / / | | |/| | | |
| * | | | | | Merge branch 'repository-files-doc' into 'master' Achilleas Pipinellis2016-08-051-0/+16
| |\ \ \ \ \ \ | | |_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add examples to repository files API ## What does this MR do? Add examples to repository files API. ## Why was this MR needed? There were no examples. ## What are the relevant issue numbers? fixes #19779 See merge request !5465
| | * | | | | Add examples to repository files API (!5465)winniehell2016-08-051-0/+16
| | | | | | |
| * | | | | | Merge branch 'date-time-to-day-tests' into 'master' Jacob Schatz2016-08-051-0/+31
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added tests to dateTime utility getDayName method ## What does this MR do? It was missing tests so i've created some tests to check the correct day name is returned See merge request !5324
| | * | | | | | Added tests to dateTime utility getDayName methoddate-time-to-day-testsPhil Hughes2016-07-181-0/+31
| | | | | | | |
| * | | | | | | Merge branch 'issuable-dropdowns-changelog' into 'master' Jacob Schatz2016-08-051-0/+3
| |\ \ \ \ \ \ \ | | |_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added changelog item for issuable form dropdowns ## What does this MR do? Adds CHANGELOG item about the new dropdowns in the issuable form See merge request !5266
View Lorry Controller Status