summaryrefslogtreecommitdiff
path: root/app
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch 'fork-to-group' into 'master'Dmitriy Zaporozhets2014-11-1813-45/+152
|\ | | | | | | | | | | | | | | | | | | | | Fork to group Fixes #1592 * project can be forked into group * fork link always lead to fork page where you select namespace where to fork See merge request !1253
| * Improve fork to namespaces featureDmitriy Zaporozhets2014-11-147-15/+78
| | | | | | | | | | | | | | | | * Show namespace thumbnail differently if project was already forked * Show loading spinner when click on fork * Fork link navigates to personal namespace only if no manageable groups exists Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * Select namespace where to fork projectDmitriy Zaporozhets2014-11-137-36/+72
| | | | | | | | | | | | | | Now you can fork project into group or personal namespace. Also I moved fork logic from ProjectsController to own fork resource Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * Refactor project fork serviceDmitriy Zaporozhets2014-11-131-3/+6
| | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * Merge branch 'prepare-fork-to-given-namespace' of ↵Dmitriy Zaporozhets2014-11-131-7/+12
| |\ | | | | | | | | | https://gitlab.com/bkaindl/gitlab-ce into bkaindl/gitlab-ce-prepare-fork-to-given-namespace
| | * Prepare ForkService to support forking projects to given namespacesBernhard Kaindl2014-10-031-7/+12
| | | | | | | | | | | | | | | | | | | | | Remove overload of BaseService.initialize, so initialize gains params, which is used to pass the namespace (like e.g. in TransferService). The namespace is checked for permission to create projects in it.
* | | Merge branch 'sidekiq_gc_start' into 'master'Dmitriy Zaporozhets2014-11-141-0/+3
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Run 'GC.start' after every EmailsOnPushWorker job This change removed a lot of large peaks of the Sidekiq memory growth graph on gitlab.com. See merge request !1257
| * | | Run 'GC.start' after every EmailsOnPushWorker jobJacob Vosmaer2014-11-141-0/+3
| | | |
* | | | Merge branch 'author_attributes_clean' into 'master'Dmitriy Zaporozhets2014-11-141-2/+6
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | Author attributes clean See merge request !1254
| * | | | Clean the string with commit author and email.Marin Jankovski2014-11-141-2/+6
| | |/ / | |/| |
* | | | Sanitize branch name and ref nameMarin Jankovski2014-11-131-1/+4
|/ / /
* | | Merge branch 'actions_in_the_inbox' into 'master'Dmitriy Zaporozhets2014-11-123-0/+34
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Actions in the inbox Related to #1607 See merge request !1247
| * | | Set action on issue/mr creation mail.Marin Jankovski2014-11-122-12/+24
| | | |
| * | | Include the helper for mailer, add links to emails.Marin Jankovski2014-11-112-0/+2
| | | |
| * | | Create emails helper for actions links.Marin Jankovski2014-11-111-0/+20
| | | |
* | | | Prevent big amount of sql queries for push serviceDmitriy Zaporozhets2014-11-121-4/+14
| | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | | Merge branch 'commit-comments' of https://gitlab.com/jeroenj/gitlab-ce into ↵Dmitriy Zaporozhets2014-11-121-3/+17
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | jeroenj/gitlab-ce-commit-comments Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> Conflicts: CHANGELOG
| * | | | Adds comments to commits in the APIJeroen Jacobs2014-09-261-3/+17
| | | | |
* | | | | Merge branch 'add_issue_edited_timestamp' into 'master'Dmitriy Zaporozhets2014-11-122-1/+14
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add issue edited timestamp Comments already have this feature. So, I think, issues should also have it. ![Снимок экрана 2014-11-08 в 17.07.10](https://gitlab.com/uploads/vernik91/gitlab-ce/4e87a15130/%D0%A1%D0%BD%D0%B8%D0%BC%D0%BE%D0%BA_%D1%8D%D0%BA%D1%80%D0%B0%D0%BD%D0%B0_2014-11-08_%D0%B2_17.07.10.png) See merge request !235
| * | | | | Add issue edited timestampNikita Verkhovin2014-11-082-1/+14
| | | | | |
* | | | | | Merge pull request #7617 from pkill/issue-280-commit-note-notificationsDmitriy Zaporozhets2014-11-121-2/+2
|\ \ \ \ \ \ | | | | | | | | | | | | | | Issue-280 Send notifications when a note is added to a commit and author...
| * | | | | | Issue-280 Send notifications when a note is added to a commit and author is ↵Alex Elman2014-11-051-2/+2
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | a group member This fixes a bug where commit authors weren't receiving email notifications for notes added to their commits and their membership was in the group but not the project. The fix is to look up membership via the team object which accounts for both project and group members.
* | | | | | Merge branch 'fix-deleted-fork' into 'master'Dmitriy Zaporozhets2014-11-112-37/+69
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Prevent post-receive error when push to project with dead forks Fixes gitlab/gitlab-cloud#173 See merge request !1246
| * | | | | | Refactor merge request refresh logic on pushDmitriy Zaporozhets2014-11-112-37/+69
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | | | | Prevent post-receive error when push to project with dead forksDmitriy Zaporozhets2014-11-111-3/+3
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If project has open merge request from fork and this fork was removed before merge request was closed it cause exception during push Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | | | | Merge pull request #7944 from cirosantilli/new-blob-commit-belowDmitriy Zaporozhets2014-11-111-2/+2
|\ \ \ \ \ \ | |/ / / / / |/| | | | | Move new blob commit message textarea below editor
| * | | | | Move new blob commit message textarea below editorCiro Santilli2014-10-131-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - match edit blob view - you enter the commit message *after* you make the modifications
* | | | | | Merge branch 'remove_unused_javascript' into 'master'Dmitriy Zaporozhets2014-11-103-7/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove unused javascript projects:teams:members:index doesn't exists See merge request !234
| * | | | | | remove unused jsskv2014-11-063-7/+1
| | | | | | |
* | | | | | | Merge branch 'better-mr-instructions' into 'master'Dmitriy Zaporozhets2014-11-101-3/+3
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Better merge request manual instructions I noticed the current instructions for manually merging a merge request that was submitted from a fork could create unnecessary merge requests or strange history. Imagine this: 1. Alice creates a repository and commits / pushes a single commit with SHA1 `A` to `master`. 2. Bob forks this repository and creates a new branch `myfeature` on `master` (`A`) 3. Alice commits `B` to `master` (which has the parent `A`) 4. Bob creates two commits on `myfeature` `P` and `Q`. 5. Bob submits the merge request to merge his `myfeature` into Alice's `master` branch. 6. Alice follows the manual merge request instructions: 1. `git checkout -b bob/repo-myfeature master` 2. `git pull http://... myfeature` The branch `bob/repo-myfeature` was created from Alice's current `master`, which was `B`. When the `pull` is executed, git will fetch Bob's branch and then merged the fetched branch `Q` into the current branch's location `B`. This creates an unnecessary merge commit from `master` into the branch Alice is trying to merge. No harm is done, but the history is a bit messier. This is even worse if Alice has set `git pull` to rebase by default. In this case, the commit `B` is rebased on top of `Q`. When Alice checks out `master` and merges in the branch, there will actually be a duplicate `B` commit. These new instructions instead tell the user to fetch Bob's `myfeature` branch. This will fetch the necessary commits `P` and `Q` and create a temporary ref `FETCH_HEAD` pointing to `Q`. Alice will then create her local `bob/repo-myfeature` branch starting at `FETCH_HEAD`. No unnecessary merge commits, and no accidental rebasing. See merge request !16
| * | | | | | | Instruct users to fetch merge request branchJohann MacDonagh2014-02-081-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Instructing users to create a new branch on the target branch and then pulling creates a few issues. If the target branch has moved on since the source branch diverged from it, then the pull will create an unnecessary merge commit from the target branch to the source branch. If the user has pull.rebase set to "true" or "preserve", then this creates an even stranger history. These instructions will ensure the local branch created for the merge request is exactly what contributing user has pushed.
* | | | | | | | make repo name link to repo homepageBen Bodenmiller2014-11-091-2/+2
| |_|_|_|/ / / |/| | | | | |
* | | | | | | Fix attachment misaligned in commentDmitriy Zaporozhets2014-11-071-1/+1
| |/ / / / / |/| | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | | | | Merge branch 'improve-white-code-highlight' into 'master'Dmitriy Zaporozhets2014-11-061-0/+8
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Improve white code highlight in comments and wiki ![Screenshot 2014-11-05 16.36.55](https://dev.gitlab.org/uploads/dzaporozhets/gitlabhq/5723848593/Screenshot_2014-11-05_16.36.55.png) See merge request !1241
| * \ \ \ \ \ Merge branch 'master' into improve-white-code-highlightDmitriy Zaporozhets2014-11-063-6/+4
| |\ \ \ \ \ \
| * | | | | | | Light gray bg for white code scheme if used in comments and wikiDmitriy Zaporozhets2014-11-051-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | | | | | | Merge branch 'master' into mentioned_inside_mrDmitriy Zaporozhets2014-11-0624-35/+17
|\ \ \ \ \ \ \ \ | | |/ / / / / / | |/| | | | | |
| * | | | | | | Merge pull request #8258 from cirosantilli/factor-repo-pathDmitriy Zaporozhets2014-11-062-4/+2
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Factor using Repository#path_to_repo
| | * | | | | | | Factor using Repository#path_to_repoCiro Santilli2014-11-052-4/+2
| | | |_|_|/ / / | | |/| | | | |
| * | | | | | | Merge branch 'gitlab-ci-parallel' into 'master'Dmitriy Zaporozhets2014-11-051-2/+2
| |\ \ \ \ \ \ \ | | |_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update GitLab CI service to work with new GitLab CI See merge request !1239
| | * | | | | | Update GitLab CI service to work with new GitLab CIDmitriy Zaporozhets2014-11-051-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | | | | | Merge branch 'auth_duplication' into 'master'Dmitriy Zaporozhets2014-11-0514-15/+0
| |\ \ \ \ \ \ \ | | |_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove unnecessary authorisations Because we already check read_project ability in `before_filter :project` See merge request !1240
| | * | | | | | remove auth duplicationValery Sizov2014-11-0514-15/+0
| | | | | | | |
| * | | | | | | Merge pull request #8094 from cirosantilli/rm-unused-authenticate_userValeriy Sizov2014-11-051-2/+0
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Remove unused authenticate_user from project#show
| | * | | | | | | Remove unused authenticate_user from project#showCiro Santilli2014-10-191-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Redundant with the authorize_read_project! filter
| * | | | | | | | Merge pull request #8233 from cirosantilli/rm-new-branch-deadValeriy Sizov2014-11-051-4/+0
| |\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Remove dead Event#new_branch? method
| | * | | | | | | | Remove dead Event#new_branch? methodCiro Santilli2014-11-031-4/+0
| | | | | | | | | |
| * | | | | | | | | Merge pull request #8234 from cirosantilli/factor-0000Valeriy Sizov2014-11-051-3/+3
| |\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Factor '0' * 40 blank ref constants [failure unrelated]
| | * | | | | | | | | Factor '0' * 40 blank ref constantsCiro Santilli2014-11-031-3/+3
| | |/ / / / / / / /
| * | | | | | | | | Merge pull request #7863 from cirosantilli/rm-blob-link-to-buttonValeriy Sizov2014-11-051-1/+2
| |\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Transform remove blob link into button.