summaryrefslogtreecommitdiff
path: root/app/services
Commit message (Collapse)AuthorAgeFilesLines
* Add system note on description change of issue/merge requestblackst0ne2017-05-042-0/+25
|
* Merge branch ↵Rémy Coutable2017-05-031-1/+4
|\ | | | | | | | | | | | | | | | | | | '29673-500-internal-server-error-when-enabling-a-deploy-key-more-than-once-through-api' into 'master' Detect already enabled DeployKeys in EnableDeployKeyService Closes #29673 See merge request !11010
| * Detect already enabled DeployKeys in EnableDeployKeyServiceMark Fletcher2017-05-031-1/+4
| | | | | | | | Ensures deploy keys can't be re-added, which causes a validation error
* | Merge branch '12910-personal-snippets-notes-show' into 'master' Sean McGivern2017-05-021-1/+1
|\ \ | | | | | | | | | | | | Display comments for personal snippets See merge request !10974
| * | Display comments for personal snippets12910-personal-snippets-notes-showJarka Kadlecova2017-05-021-1/+1
| |/
* | Merge branch '26488-target-disabled-mr' into 'master' Sean McGivern2017-05-021-1/+1
|\ \ | |/ |/| | | | | | | | | Fix 404 when upstream disabled merge requests Closes #26488 See merge request !10427
| * Fix 404 when upstream has disabled merge requestsmhasbini2017-04-271-1/+1
| |
* | Revert "Merge branch 'tc-no-todo-service-select' into 'master'"revert-c3c465acStan Hu2017-04-292-11/+7
| | | | | | This reverts merge request !10845
* | Merge branch 'tc-no-todo-service-select' into 'master' Robert Speicher2017-04-282-7/+11
|\ \ | | | | | | | | | | | | | | | | | | Avoid plucking Todo ids in TodoService Closes #30374 See merge request !10845
| * | Avoid plucking Todo ids and use sub-queries insteadtc-no-todo-service-selectToon Claes2017-04-272-7/+11
| |/ | | | | | | | | | | | | | | TodoService should not call `.select(&:id)` on todos, because this is bad performance. So instead use sub-queries, which will result in a single SQL query to the database. https://docs.gitlab.com/ee/development/sql.html#plucking-ids
* | Add board_move slash commandAlex Sanford2017-04-282-1/+23
| |
* | Make namespace owner master of project upon creationtc-make-user-master-project-by-adminToon Claes2017-04-271-1/+2
|/ | | | | When an admin creates a project in the namespace of a user, that user automatically gains master access to that project.
* Fix logic in Users::CreateService broken by the fix for OAuth usersRémy Coutable2017-04-261-8/+15
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Ensures that OAuth/LDAP/SAML users don't need to be confirmedRémy Coutable2017-04-261-4/+4
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Fix OAuth, LDAP and SAML SSO when regular sign-ups are disableddm-fix-oauth-user-creationDouwe Maan2017-04-242-4/+4
|
* Merge branch 'tc-realtime-every-pipeline-on-mr' into 'master' Kamil Trzciński2017-04-241-51/+0
|\ | | | | | | | | | | | | Properly expire cache for **all** MRs of a pipeline Closes #31040 See merge request !10770
| * Properly expire cache for **all** MRs of a pipelineToon Claes2017-04-241-51/+0
| | | | | | | | | | Turn ExpirePipelineCacheService into Worker so it can fetch all the merge requests for which the pipeline runs or did run against.
* | Move records to the ghost user in a transaction.Timothy Andrew2017-04-241-11/+23
|/ | | | | | | | | | | | | | | | - While deleting a user, some of the user's associated records are moved to the ghost user so they aren't deleted. The user is blocked before these records are moved, to prevent the user from creating new records while the migration is happening, and so preventing a data race. - Previously, if the migration failed, the user would _remain_ blocked, which is not the expected behavior. On the other hand, we can't just stick the block + migration into a transaction, because we want the block to be committed before the migration starts (for the data race reason mentioned above). - One solution (implemented in this commit) is to block the user in a parent transaction, migrate the associated records in a nested sub-transaction, and then unblock the user in the parent transaction if the sub-transaction fails.
* Merge branch 'dz-refactor-create-members' into 'master' Douwe Maan2017-04-201-1/+7
|\ | | | | | | | | Refactor code that creates and destroys project/group members See merge request !10735
| * Refactor controller code that creates project/group membersDmitriy Zaporozhets2017-04-171-1/+7
| | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | Merge branch 'uassign_on_member_removing' into 'master' Sean McGivern2017-04-201-1/+23
|\ \ | | | | | | | | | | | | | | | | | | Unassign all Issues and Merge Requests when member leaves a team Closes #30768 and #24117 See merge request !10755
| * | Remove helpers assigned_issuables_count and cached_assigned_issuables_countuassign_on_member_removingValery Sizov2017-04-201-0/+1
| | |
| * | Unassign all Issues and Merge Requests when member leaves teamValery Sizov2017-04-181-1/+22
| |/
* | Refactor changing files in web UIDouwe Maan2017-04-2011-302/+125
| |
* | Merge branch 'sh-issue-29247-fix' into 'master' Douwe Maan2017-04-191-0/+11
|\ \ | | | | | | | | | | | | | | | | | | Don't delete a branch involved in an open merge request in "Delete all merged branches" service Closes #29427 See merge request !10727
| * | Don't delete a branch involved in an open merge request in "Delete all ↵sh-issue-29247-fixStan Hu2017-04-191-0/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | merged branches" service Customers were surprised by the previous behavior, which destroyed branches even though an open merge request existed for it. Closes #29427
* | | Merge branch 'usage-ping-port' into 'master' Sean McGivern2017-04-193-0/+124
|\ \ \ | |/ / |/| | | | | | | | | | | | | | Usage ping port Closes #27750 See merge request !10481
| * | Use last_activity_on in cohortsSean McGivern2017-04-141-6/+6
| | |
| * | Use serializer for formatting cohorts dataSean McGivern2017-04-141-27/+40
| | |
| * | Rename user cohorts -> cohortsSean McGivern2017-04-141-1/+1
| | |
| * | Fix column name in user cohortsSean McGivern2017-04-141-1/+1
| | |
| * | Make UserCohortsService more understandableSean McGivern2017-04-141-34/+72
| | | | | | | | | | | | | | | 1. Extract out into several methods. 2. Add more comments describing the data and the shape of the data.
| * | Add user cohorts table to admin areaSean McGivern2017-04-141-0/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This table shows the percentage of users who registered in the last twelve months, who last signed in during or later than each of those twelve months, by month. It is only enabled when the usage ping is enabled, and the page also shows pretty-printed usage ping data. The cohorts table is generated in Ruby from some basic SQL queries, because performing the gap-filling and running sums needed in both MySQL and Postgres is painful.
| * | Add a new Gitlab::UserActivities class to track user activitiesRémy Coutable2017-04-141-1/+1
| | | | | | | | | | | | | | | | | | This new class uses a Redis Hash instead of a Sorted Set. Signed-off-by: Rémy Coutable <remy@rymai.me>
| * | Remove user activities table and use redis instead of PG for recording ↵James Lopez2017-04-141-5/+1
| | | | | | | | | | | | | | | | | | activities Refactored specs and added a post deployment migration to remove the activity users table.
| * | Add user activity service and spec. Also added relevant - NOT offline - ↵James Lopez2017-04-142-0/+28
| |/ | | | | | | | | | | | | migration It uses a user activity table instead of a column in users. Tested with mySQL and postgreSQL
* | Refactor group search out of global searchNick Thomas2017-04-183-7/+24
| |
* | Does not remove the GitHub remote when importing from GitHubDouglas Barbosa Alexandre2017-04-181-1/+0
| |
* | Hard delete users' associated records deleted from AbuseReportssh-add-option-to-hard-deleteStan Hu2017-04-161-1/+1
|/ | | | | | In the case of spammers, we really want a hard delete to avoid retaining spam. Closes #31021
* Merge branch '30349-create-users-build-service' into 'master' Rémy Coutable2017-04-142-92/+103
|\ | | | | | | | | | | | | Implement Users::BuildService Closes #30349 See merge request !10675
| * Implement Users::BuildServicegeoandri2017-04-132-92/+103
| |
* | Improve error messages when a record is unable to be created for a projectStan Hu2017-04-131-0/+3
|/
* Remove the User#is_admin? methodblackst0ne2017-04-091-3/+3
|
* Merge branch 'master' into new-resolvable-discussionLuke "Jared" Bennett2017-04-076-7/+27
|\
| * Merge branch 'bvl-fix-project-ci-status-cache' into 'master' Kamil Trzciński2017-04-071-0/+2
| |\ | | | | | | | | | | | | Fix invalidating Project build status cache to often See merge request !10313
| | * Use `Ci::ExpirePipelineCacheService` to set `ProjectPipelinestatus`bvl-fix-project-ci-status-cacheBob Van Landuyt2017-04-071-0/+2
| | |
| * | Merge branch '18471-restrict-tag-pushes-protected-tags' into 'master' Douwe Maan2017-04-075-7/+25
| |\ \ | | |/ | |/| | | | | | | | | | | | | Protected Tags Closes #18471 See merge request !10356
| | * Merge branch 'master' into '18471-restrict-tag-pushes-protected-tags'Kushal Pandya2017-04-064-40/+114
| | |\ | | | | | | | | | | | | # Conflicts: # spec/lib/gitlab/import_export/all_models.yml
| | * \ Merge branch 'last-green-master' into 18471-restrict-tag-pushes-protected-tagsJames Edwards-Jones2017-04-062-2/+2
| | |\ \
| | * | | Protected Tags backend review changesJames Edwards-Jones2017-04-062-10/+4
| | | | | | | | | | | | | | | Added changelog
View Lorry Controller Status