summaryrefslogtreecommitdiff
path: root/spec
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | | Merge branch 'ldap-special-chars-fix' into 'master' Douwe Maan2015-12-241-0/+15
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix identity and user retrieval when special characters are used Fixes #4023 I also added tests to make sure the user with special characters in his name is returned correctly. @rspeicher this probably should be added to 8.3 as a patch. See merge request !2176
| * | | | | | | | Fix identity and user retrieval when special characters are usedPatricio Cano2015-12-221-0/+15
| | | | | | | | |
* | | | | | | | | Fix NewBranchForm specDouwe Maan2015-12-241-1/+1
| |_|_|/ / / / / |/| | | | | | |
* | | | | | | | Merge branch 'fix-milestones-with-slashes' into 'master' Dmitriy Zaporozhets2015-12-241-0/+10
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix Error 500 when global milestones have slashes * Closes #4226 * Closes https://github.com/gitlabhq/gitlabhq/issues/9921 See merge request !2182
| * | | | | | | | Fix Error 500 when global milestones have slashesStan Hu2015-12-221-0/+10
| |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | Closes #4226
* | | | | | | | Merge branch 'edit-on-fork' into 'master' Dmitriy Zaporozhets2015-12-241-4/+4
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatically fork a project when not allowed to edit a file. Fixes #3215. To do: - [ ] Add tests ----- ## "Edit" button on file in a project the user does NOT have write access to ![edit_file](/uploads/7602157420768aef483a6586bba2d164/edit_file.png) ## Clicking will automatically create a fork ![during_fork](/uploads/32f4f5dff9f24ea99522000b0bf881c5/during_fork.png) ## When the fork has been created, the user is returned to the edit page on the original project with a notice ![edit_notice](/uploads/94ed1319404370ff1e9c0d672fb41e03/edit_notice.png) ## The user cannot change the target branch and is informed that editing will start an MR ![edit_footer](/uploads/4da68d4795c7177e575b7c434d16eeae/edit_footer.png) ## Hitting "Commit changes" will commit and start an MR from my fork to the origin project ![Screen_Shot_2015-12-17_at_23.38.08](/uploads/d777a4db6f38a5a1be84031694465bc1/Screen_Shot_2015-12-17_at_23.38.08.png) ----- ## "Create file, "Upload file" and "New directory" buttons in a project the user does NOT have write access to ![new_directory](/uploads/72f556248f30d6652523bbb4be01b3e0/new_directory.png) ## Clicking any of these options will automatically create a fork ![during_fork](/uploads/32f4f5dff9f24ea99522000b0bf881c5/during_fork.png) ## When the fork has been created, the user is returned to the tree page on the original project with a notice ![new_directory_notice](/uploads/a1a3e11308ae0e8f0913fae6813a37ed/new_directory_notice.png) ## Clicking "New directory" again will show the modal. The user cannot change the target branch and is informed that editing will start an MR ![new_dir](/uploads/99ca8cbfb2f70603e352b3fdf67b6281/new_dir.png) ## Hitting "Create directory" will commit and start an MR from my fork to the origin project ![Screen_Shot_2015-12-17_at_23.39.19](/uploads/3713d0235abf831361b803a6198c5bc1/Screen_Shot_2015-12-17_at_23.39.19.png) cc @dzaporozhets @skyruler See merge request !2145
| * | | | | | | | Fix specsDouwe Maan2015-12-181-2/+2
| | | | | | | | |
| * | | | | | | | Automatically fork a project when not allowed to edit a file.Douwe Maan2015-12-181-2/+2
| | |_|_|/ / / / | |/| | | | | |
* | | | | | | | Merge branch 'fix/visibility-level-setting-in-forked-projects' into 'master' Douwe Maan2015-12-242-0/+63
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix/visibility level setting in forked projects Fixes #3136 /cc @DouweM See merge request !1744
| * | | | | | | | Add some specs for forked project visibility_level casesTomasz Maczukin2015-12-242-0/+63
| | |_|_|/ / / / | |/| | | | | |
* | | | | | | | Merge branch 'branch-invalid-name' into 'master' Douwe Maan2015-12-243-1/+165
|\ \ \ \ \ \ \ \ | |_|_|_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add JS validation for invalid characters in branch name Fixes #3293 Demo: ![out-1080p](/uploads/ba21c359b6b8b440c40cacf772ec0df7/out-1080p.gif) See merge request !2122
| * | | | | | | Add helper methods to JS validation spec for invalid branch namesbranch-invalid-nameDouglas Barbosa Alexandre2015-12-221-87/+92
| | | | | | | |
| * | | | | | | Fix minor stylistic complaintsDouglas Barbosa Alexandre2015-12-211-34/+34
| | | | | | | |
| * | | | | | | Fix specDouglas Barbosa Alexandre2015-12-181-1/+1
| | | | | | | |
| * | | | | | | Merge branch 'master' into branch-invalid-nameDouglas Barbosa Alexandre2015-12-182-4/+23
| |\ \ \ \ \ \ \ | | | |/ / / / / | | |/| | | | |
| * | | | | | | Add specs for JS validation for invalid characters in branch nameDouglas Barbosa Alexandre2015-12-172-0/+159
| | | | | | | |
* | | | | | | | Merge branch 'emoji-picker-fix' into 'master' Valery Sizov2015-12-241-2/+2
|\ \ \ \ \ \ \ \ | |_|_|_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Emoji picker: better alias handling related to https://gitlab.com/gitlab-org/gitlab-ce/issues/3576 See merge request !2198
| * | | | | | | fix specemoji-picker-fixValery Sizov2015-12-241-2/+2
| | | | | | | |
* | | | | | | | Merge branch 'add-project-permissions' into 'master' Dmitriy Zaporozhets2015-12-231-0/+13
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add project permissions to all project API endpoints This standardizes all the project API formats. Also needed to support Huboard. See merge request !2090
| * | | | | | | | Add project permissions to all project API endpointsStan Hu2015-12-221-0/+13
| | |_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | This standardizes all the project API formats. Also needed to support Huboard.
* | | | | | | | Merge branch 'fix/ci-runners-token-persistence' into 'master' Grzegorz Bizon2015-12-233-8/+18
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix method that ensures authentication token Until now, `ensure_#{token_filed_name}!` method didn't persist new token in database. This closes #4235. See merge request !2185
| * | | | | | | Use method that creates runners registration tokenGrzegorz Bizon2015-12-232-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `runners_registration_token` now creates a new token if it is blank.
| * | | | | | | Add method that persist ensured token in `TokenAuthenticatable`Grzegorz Bizon2015-12-231-6/+13
| | | | | | | |
| * | | | | | | Fix method that ensures authentication tokenGrzegorz Bizon2015-12-231-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Until now, `ensure_#{token_filed_name}` method didn't persist new token in database. This closes #4235.
* | | | | | | | Merge branch 'improve_emoji_picker' into 'master' Dmitriy Zaporozhets2015-12-231-12/+0
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Improve emoji picker This request uses an emoji image sprite to show big set of emojis. The sprite has been made based one gemojione gem images. The categories are retrieved from https://github.com/jonathanwiesel/gemojione/blob/master/config/index.json as well. Some categories are pretty poor, it can be extended in the future. It's a first step on the way to improve picker. The first implementation was using https://github.com/iamcal/emoji-data project, but there was huge incompatibility with gemojione emojis set and I had to drop that solution. ![Screenshot_2015-12-22_23.26.27](/uploads/7d323eb0586204ac92915b41233b97ec/Screenshot_2015-12-22_23.26.27.png) See merge request !2172
| * | | | | | | | fix specsValery Sizov2015-12-221-6/+0
| | | | | | | | |
| * | | | | | | | base implementation of emoji picker [ci skip]Valery Sizov2015-12-221-6/+0
| |/ / / / / / /
* | | | | | | | Merge branch 'fix/ci-lint' into 'master' Grzegorz Bizon2015-12-232-28/+39
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make CI Lint form synchronous This removes `remote: true` from CI Lint form, making it synchronous form. This also removes some complexity related to displaying lint messages. View also has been updated, removed deprecated Bootstrap 2 tags. Improved design. See merge request !2171
| * | | | | | | Make CI Lint form synchronousfix/ci-lintGrzegorz Bizon2015-12-221-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This removes `remote: true` from CI Lint form, making it synchronous form. This also removes some complexity related to displaying lint messages. View also has been updated, removed deprecated Bootstrap 2 tags. Improved design. Closes #4206
| * | | | | | | Rename CI lint specs fileGrzegorz Bizon2015-12-221-0/+0
| | | | | | | |
| * | | | | | | Improve CI Lint specs (refactoring)Grzegorz Bizon2015-12-221-18/+29
| |/ / / / / /
* | | | | | | Merge branch 'fix_leave_project_display' into 'master' Stan Hu2015-12-222-0/+24
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix and test leave project display The 'Leave this project' display on project page was broken. It was calling the method incorrectly, which returned `nil` and displayed nothing. This fixes the display and also adds 2 different tests for it. cc/ @jvanbaarsen @stanhu Please review and merge if you think it looks good. Thank you! See merge request !2166
| * | | | | | Fix and test leave project displayDrew Blessing2015-12-212-0/+24
| | | | | | |
* | | | | | | Move `ci_build_details_path` helper spec to correct locationRobert Speicher2015-12-212-22/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Also, make it not fail.
* | | | | | | removes api credentials from link to build_pageMarvin Frick2015-12-221-0/+22
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | Also adds a spec for MergeRequestHelper to avoid having a regression later on.
* | | | | | Merge branch 'add_email_unlock' into 'master' Robert Speicher2015-12-211-0/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow account unlock via email We see a lot of users get confused about what it means when your account gets locked. Many try to reset their password and are still faced with a lockout. With this change, users receive an email that allows them to unlock their account immediately. The previous behavior where the account is auto-unlocked after a time also still works. See merge request !2049
| * | | | | | Allow account unlock via emailDrew Blessing2015-12-141-0/+1
| | | | | | |
* | | | | | | Merge branch 'issue_3452'Douwe Maan2015-12-211-0/+64
|\ \ \ \ \ \ \
| * | | | | | | Fix broken spec. #3452Rubén Dávila2015-12-161-1/+1
| | | | | | | |
| * | | | | | | Add link to MR from Build detail page. #3452Rubén Dávila2015-12-161-0/+64
| | | | | | | |
* | | | | | | | Merge branch 'coverage-regex' into 'master' Douwe Maan2015-12-211-0/+6
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix build coverage regex matching to allow captures. Fixes #2644 /cc @DouweM See merge request !2138
| * | | | | | | | Fix build coverage regex.Jared Szechy2015-12-201-0/+6
| | |_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added a spec for regex captures as well. Fixes #2644
* | | | | | | | Backport JIRA serviceDrew Blessing2015-12-189-9/+391
| |_|_|_|/ / / |/| | | | | |
* | | | | | | Fix feature specs: we always show the build status if ci_commit is presentci-commit-status-skippedKamil Trzcinski2015-12-171-23/+6
| | | | | | |
* | | | | | | Don't create CI status for refs that doesn't have .gitlab-ci.yml, even if ↵Kamil Trzcinski2015-12-171-6/+11
|/ / / / / / | | | | | | | | | | | | | | | | | | the builds are enabled
* | | | | | Merge branch 'api-support-starred-projects' into 'master' Dmitriy Zaporozhets2015-12-161-1/+20
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Api support for requesting starred projects for user Closes #4112 Note: probably targets the wrong release in the `CHANGELOG`, though 8.4 was not there yet See merge request !2127
| * | | | | | Api support for requesting starred projects for userZeger-Jan van de Weg2015-12-161-1/+20
| | |_|/ / / | |/| | | | | | | | | | | | | | | | Fixes #4112
* | | | | | Merge branch 'master' into fix-merge-immediately-buttonDouwe Maan2015-12-161-8/+3
|\ \ \ \ \ \ | |/ / / / /
| * | | | | Merge remote-tracking branch 'origin/rs-simplify-status-icon'Dmitriy Zaporozhets2015-12-161-8/+3
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| | * | | | | Define CI status icon colors in SCSS instead of a helperrs-simplify-status-iconRobert Speicher2015-12-121-8/+3
| | | | | | |