summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Split to two commands, feedback:test-db-rollbackLin Jen-Shin2017-03-141-1/+2
| | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/9908#note_25331127
* Use `remove_foreign_key :timelogs, name: '...'`Lin Jen-Shin2017-03-141-9/+2
| | | | | Feedback: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/9908#note_25324225
* Drop the index only for postgresql, becauseLin Jen-Shin2017-03-141-1/+2
| | | | | | mysql cannot simply drop the index without dropping the corresponding foreign key, and we certainly don't want to drop the foreign key here.
* Disable rubocop for down methodLin Jen-Shin2017-03-141-0/+1
|
* Fix for postgresqlLin Jen-Shin2017-03-144-4/+12
|
* Add a test which would rollback db and migrate againLin Jen-Shin2017-03-149-7/+26
| | | | Closes #29106
* Merge branch '28385-27189-tech-debt-dropdown' into 'master' Fatih Acet2017-03-131-1/+1
|\ | | | | | | | | Simplify jQuery selector See merge request !9881
| * Simplify selector28385-27189-tech-debt-dropdownFilipa Lacerda2017-03-101-1/+1
| |
* | Merge branch 'refactor/global-permissions-for-internal-users' into 'master' Sean McGivern2017-03-1321-65/+105
|\ \ | | | | | | | | | | | | Refactor/global permissions for internal users See merge request !9598
| * | allow internal attributes to have a nil valuerefactor/global-permissions-for-internal-usershttp://jneen.net/2017-03-101-1/+1
| | |
| * | mark internal users in the admin panelhttp://jneen.net/2017-03-091-1/+3
| | | | | | | | | | | | and don't offer to impersonate them
| * | reverse the logic and use a clearer namehttp://jneen.net/2017-03-091-7/+7
| | |
| * | get the logic right :Xhttp://jneen.net/2017-03-091-1/+1
| | |
| * | add .internal and .non_internal scopeshttp://jneen.net/2017-03-091-1/+13
| | |
| * | consolidate the error handling for #impersonatehttp://jneen.net/2017-03-091-9/+12
| | |
| * | add a spec that the ghost user cannot log inhttp://jneen.net/2017-03-091-0/+12
| | |
| * | spec the new behavior of .class_forhttp://jneen.net/2017-03-091-4/+6
| | | | | | | | | | | | and more robustly spec the ancestor behavior
| * | spec the behavior of nil subjectshttp://jneen.net/2017-03-091-0/+6
| | |
| * | fix a brittle stubhttp://jneen.net/2017-03-091-1/+1
| | | | | | | | | | | | | | | true is neither nil nor a user and doesn't make sense as the return value of `current_user`
| * | use policies to protect sending emailhttp://jneen.net/2017-03-092-1/+2
| | |
| * | don't require ghost users to be blockedhttp://jneen.net/2017-03-092-23/+0
| | |
| * | don't require passwords for internal usershttp://jneen.net/2017-03-091-1/+8
| | | | | | | | | | | | since they can't log in
| * | protect git access through the policy infrahttp://jneen.net/2017-03-091-1/+1
| | |
| * | protect internal users from impersonationhttp://jneen.net/2017-03-091-0/+4
| | |
| * | use the policy stack to protect loginshttp://jneen.net/2017-03-094-7/+6
| | |
| * | add User#internal? and some global permissionshttp://jneen.net/2017-03-092-1/+10
| | |
| * | use a magic default :global symbol instead of nilhttp://jneen.net/2017-03-0910-15/+21
| | | | | | | | | | | | to make sure we mean the global permissions
* | | Merge branch 'backport-cache-middleware-fix' into 'master' Rémy Coutable2017-03-131-0/+24
|\ \ \ | | | | | | | | | | | | | | | | Backport cache middleware fix See merge request !9867
| * | | Try out the monkey patch to fix the middlewarebackport-cache-middleware-fixLin Jen-Shin2017-03-111-0/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | which isn't properly cleaning up whenever something is thrown, not raised. See: https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/1402#note_25128108
* | | | Merge branch '24137-issuable-permalink' into 'master' Rémy Coutable2017-03-134-1/+43
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | link issuable reference to itself in header Closes #24137 See merge request !9641
| * | | | link issuable reference to itself in headermhasbini2017-03-094-1/+43
| | | | |
* | | | | Merge branch 'fix_updated_field_in_atom_files' into 'master' Rémy Coutable2017-03-136-5/+9
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix xml.updated field in rss/atom feeds Closes #19437 See merge request !9889
| * | | | | Fix xml.updated field in rss/atom feedsblackst0ne2017-03-126-5/+9
| | | | | |
* | | | | | Merge branch '29189-discussion-button' into 'master' Filipa Lacerda2017-03-133-15/+17
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix alignment of resolve discussion button Closes #29189 See merge request !9865
| * | | | | | Fix alignment of resolve discussion button29189-discussion-buttonAnnabel Dunstone Gray2017-03-103-15/+17
| | | | | | |
* | | | | | | Merge branch '29328-fix-transient-failure-in-model-user-spec' into 'master' Rémy Coutable2017-03-132-2/+7
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Resolve transient failure in spec/models/user_spec.rb Closes #29328 See merge request !9892
| * | | | | | | Resolve transient failure in spec/models/user_spec.rbGeorge Andrinopoulos2017-03-132-2/+7
|/ / / / / / /
* | | | | | | Merge branch 'mrchrisw-update-deploy-key' into 'master' Filipa Lacerda2017-03-131-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix inconsistent deploy key documentation in UI See merge request !9817
| * | | | | | | Fix inconsistent deploy key documentation in UImrchrisw-update-deploy-keyChris Wilson2017-03-081-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Deploy keys were added with write access to https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7383 - We still state " Deploy keys allow read-only access to your repository." in the UI. This updates the deploy key UI information to reflect the docs https://docs.gitlab.com/ce/ssh/README.html#deploy-keys
* | | | | | | | Merge branch 'zj-update-templates' into 'master' James Lopez2017-03-135-29/+29
|\ \ \ \ \ \ \ \ | |_|_|/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | Update templates with new CI variables See merge request !9852
| * | | | | | | Update templates with new CI variableszj-update-templatesZ.J. van de Weg2017-03-105-29/+29
| | | | | | | |
* | | | | | | | Merge branch 'fix/execute-inflections-initializer-early' into 'master' DJ Mountney2017-03-111-0/+0
|\ \ \ \ \ \ \ \ | |_|_|_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | Execute inflections initializer earlier See merge request !9871
| * | | | | | | Execute inflections initializer earlierfix/execute-inflections-initializer-earlyDouglas Barbosa Alexandre2017-03-101-0/+0
| | |_|/ / / / | |/| | | | |
* | | | | | | Merge branch '29046-fix-github-importer-open-prs' into 'master' Douwe Maan2017-03-105-33/+105
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix GitHub Import deleting branches for open PRs from a fork Closes #29046 See merge request !9758
| * | | | | | | Using guard clause and added more specs29046-fix-github-importer-open-prsGabriel Mazetto2017-03-104-58/+90
| | | | | | | |
| * | | | | | | ChangelogGabriel Mazetto2017-03-101-0/+4
| | | | | | | |
| * | | | | | | Fix GitHub Import for open PRs from a forkGabriel Mazetto2017-03-103-1/+37
| |/ / / / / /
* | | | | | | Merge branch 'add-frequently-used-emojis-back-to-menu' into 'master' Filipa Lacerda2017-03-103-23/+61
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | | | | | | | | | | | | | | | Add frequently used emojis back to awards menu See merge request !9843
| * | | | | | Add frequently used emojis back to awards menuadd-frequently-used-emojis-back-to-menuEric Eastwood2017-03-093-23/+61
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | Thanks @filipa for the shout`
* | | | | | Merge branch 'fix-500-in-notes-polling' into 'master' Rémy Coutable2017-03-101-1/+1
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix notes polling failing after code changes Closes #29327 See merge request !9862