- 06 Apr, 2017 26 commits
-
-
Douwe Maan authored
Resolve "Deleting a user shouldn't delete associated records" See merge request !1581
-
Filipa Lacerda authored
Open new board after creating it Closes gitlab-ce#29611 See merge request !1449
-
Timothy Andrew authored
1. Have `MigrateToGhostUser` be a service rather than a mixed-in module, to keep things explicit. Specs testing the behavior of this class are moved into a separate service spec file. 2. Add a `user.reported_abuse_reports` association to make the `migrate_abuse_reports` method more consistent with the other `migrate_` methods.
-
Rémy Coutable authored
elasticsearch: Add support for an experimental repository indexer Closes #1606 See merge request !1483
-
Jacob Schatz authored
Recent search history for issues -- EE merge edition Closes gitlab-ce#27262 See merge request !1547
-
Alfredo Sumaran authored
Linking to edit file directly -- EE merge edition See merge request !1546
-
Eric Eastwood authored
-
Eric Eastwood authored
Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/27262 Conflicts: app/assets/javascripts/filtered_search/filtered_search_manager.js app/views/shared/issuable/_search_bar.html.haml
-
Nick Thomas authored
-
Timothy Andrew authored
-
Timothy Andrew authored
-
Timothy Andrew authored
... when the user is destroyed. 1. Normally, for a given awardable and award emoji name, a user is only allowed to create a single award emoji. 2. This validation needs to be removed for ghost users, since: - User A and User B have created award emoji - with the same name and against the same awardable - User A is deleted. Their award emoji is moved to the ghost user - User B is deleted. Their award emoji needs to be moved to the ghost user. However, this breaks the uniqueness validation, since the ghost user is only allowed to have one award emoji of a given name for a given awardable
-
Timothy Andrew authored
... when the user is destroyed. To clarify, this regards abuse reports that the to-be-deleted user has _reported_.
-
Timothy Andrew authored
Introduction ------------ 1. The foreign key was not explicitly specified on the association. 2. The `AbuseReport` model contains two references to user - `reporter_id` and `user_id` 3. `user.abuse_report` is supposed to return the single abuse report where `user_id` refers to the given user. Bug Description --------------- 1. `user.abuse_report` would return an abuse report where `reporter_id` referred to the current user, if such an abuse report was present. 2. This implies a slightly more serious bug as well: - Assume User A filed an abuse report against User B - We have an abuse report where `reporter_id` is User A and `user_id` is User B - If User A is updated (`user_a.block`, for example), the abuse report would also be updated, such that both `reporter_id` _and_ `user_id` point to User A. Fix --- Explicitly declare the foreign key `user_id` in the `has_one` declaration
-
Timothy Andrew authored
... when the user is destroyed.
-
Timothy Andrew authored
1. When the user is deleted. 2. Refactor out code relating to "migrating records to the ghost user" into a `MigrateToGhostUser` concern, which is tested using a shared example.
-
Rémy Coutable authored
CE Upstream - Wednesday Closes gitlab-ci-multi-runner#2284 See merge request !1577
-
Jacob Schatz authored
Add merge request discrepancy resolutions to documentation See merge request !1532
-
Rémy Coutable authored
Signed-off-by: Rémy Coutable <remy@rymai.me>
-
Stan Hu authored
Rename Rake task from geo:add_primary_node to geo:set_primary_node See merge request !1579
-
DJ Mountney authored
[ci skip]
-
DJ Mountney authored
[ci skip]
-
DJ Mountney authored
[ci skip]
-
DJ Mountney authored
[ci skip]
-
DJ Mountney authored
[ci skip]
-
DJ Mountney authored
[ci skip]
-
- 05 Apr, 2017 14 commits
-
-
Stan Hu authored
-
Rémy Coutable authored
Read track from deployment and visualise canary deployments See merge request !1551
-
Rémy Coutable authored
Strangely enough, this seems to result from the upstream merge: https://gitlab.com/gitlab-org/gitlab-ee/commit/adbfff09e3e97eaa2e77b2888c148989e7d5444c#a66b0cb7d7efb8c153da597f254084a47dae4a38Signed-off-by: Rémy Coutable <remy@rymai.me>
-
Jacob Schatz authored
Burndown Charts Closes #91 See merge request !1540
-
Rémy Coutable authored
Signed-off-by: Rémy Coutable <remy@rymai.me>
-
Kamil Trzcinski authored
-
Kamil Trzciński authored
Add check for deploy boards See merge request !1555
-
Achilleas Pipinellis authored
Fix elasticsearch docs: ID=1 sudo ... is ineffective See merge request !1573
-
Douwe Maan authored
Add a Rake task to make the current node the primary Geo node See merge request !1533
-
Robert Speicher authored
Build issues for merge request when there is an issue template Closes #1968 See merge request !1550
-
Rémy Coutable authored
Signed-off-by: Rémy Coutable <remy@rymai.me>
-
Rémy Coutable authored
Signed-off-by: Rémy Coutable <remy@rymai.me>
-
Filipa Lacerda authored
Removes the duplicated search icon in emoji menu Closes #30416 and #30016 See merge request !10472
-
Jacob Schatz authored
Render PDFs in GitLab See merge request !10419
-