summaryrefslogtreecommitdiff
path: root/spec/routing
Commit message (Collapse)AuthorAgeFilesLines
* Remove format from end of URL for URLs that take a ref or pathDouwe Maan2017-04-171-1/+1
|
* Add routing specs for environment foldersGrzegorz Bizon2017-03-271-0/+49
|
* Implement OpenID Connect identity providerMarkus Koller2017-03-071-0/+30
|
* Use native unicode emojisEric Eastwood2017-03-061-2/+1
| | | | | | | | | | | | | | | | | | | | - gl_emoji for falling back to image/css-sprite when the browser doesn't support an emoji - Markdown rendering (Banzai filter) - Autocomplete - Award emoji menu - Perceived perf - Immediate response because we now build client-side - Update `digests.json` generation in gemojione rake task to be more useful and include `unicodeVersion` MR: !9437 See issues - #26371 - #27250 - #22474
* Remove query parameters from notes polling endpoint to make caching easierAdam Niedzielski2017-03-011-4/+14
|
* Merge branch 'master' into jej-pages-picked-from-eeJames Edwards-Jones2017-02-031-4/+4
|\
| * replace `find_with_namespace` with `find_by_full_path`Adam Pahlevi2017-02-031-4/+4
| | | | | | | | add complete changelog for !8949
* | Fix 500 error while navigating to the `pages_domains` 'show' page.Timothy Andrew2017-01-311-7/+30
|/ | | | | | | | | | | | | | | | | | | | | | | | | | | | ================== = Implementation = ================== 1. The path of the page is of the form 'group/project/pages/domains/<domain_name>' 2. Rails looks at `params[:id]` (which should be the domain name), and finds the relevant model record. 3. Given a domain like `foo.bar`, Rails sets `params[:id]` to `foo` (should be `foo.bar`), and sets `params[:format]` to `bar` 4. This commit fixes the issue by adding a route constraint, so that `params[:id]` is set to the entire `foo.bar` domain name. ========= = Tests = ========= 1. Add controller specs for the `PagesDomainController`. These are slightly orthogonal to this bug fix (they don't fail when this bug is present), but should be present nonetheless. 2. Add routing specs that catch this bug (by asserting that the `id` param is passed as expected when it contains a domain name). 3. Modify the 'RESTful project resources' routing spec shared example to accomodate controllers where the controller path (such as `pages/domains`) is different from the controller name (such as `pages_domains`).
* Fix 500 error when visit group from admin area if group name contains dotdz-improve-admin-group-routingDmitriy Zaporozhets2016-12-281-2/+8
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Modify GithubImport to support GiteaRémy Coutable2016-12-191-1/+0
| | | | | | | | The reason is that Gitea plan to be GitHub-compatible so it makes sense to just modify GitHubImport a bit for now, and hopefully we can change it to GitHubishImport once Gitea is 100%-compatible. Signed-off-by: Rémy Coutable <remy@rymai.me>
* Rename Gogs to Gitea, DRY the controller and improve viewsRémy Coutable2016-12-191-0/+166
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Changed autocomplete_sources into an action that returns a single 'at' type ↵18435-autocomplete-is-not-performantLuke Bennett2016-12-151-4/+15
| | | | | | | | | | | | | | | | | | | | | | | | of sources at a time Finished up autocomplete_sources action and added frontend to fetch data only when its needed Added wait_for_ajax to specs Fixed builds and improved the setup/destroy lifecycle Changed global namespace and DRYed up loading logic Added safety for accidentally loading data twice Removed destroy as its not necessary and is messing with click events from a blur race condition Created AutocompleteSourcesController and updated routes Fixed @undefined from tabbing before load ends Disable tabSelectsMatch until we have loaded data Review changes
* Add support for nested groups to admin routingdz-fix-admin-routesDmitriy Zaporozhets2016-12-111-1/+13
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Add nested groups support to the routingdz-allow-nested-group-routingDmitriy Zaporozhets2016-11-232-510/+540
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Fix 500 error when group name ends with gitDmitriy Zaporozhets2016-11-211-1/+1
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Fix 404 on some group pages when name contains dotdz-fix-group-name-dotDmitriy Zaporozhets2016-11-211-5/+13
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Fix routing spec for group controllerdz-refactor-contraintsDmitriy Zaporozhets2016-11-081-2/+2
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Merge branch 'dz-rename-user-routes' into 'master'Robert Speicher2016-10-171-6/+6
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | Rename users routing from /u/:username to /users/:username for consistency with other routes Renames /u/:username to /users/:username To follow consistency with other routes (like groups) and UsersController name. Now when you can use `/:username` for accessing user page there is no need in shortcut like `/u/` See merge request !6851
| * Update users routing specDmitriy Zaporozhets2016-10-131-6/+6
| | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | Fix 404 when group path has dot in the namedz-fix-group-with-dotDmitriy Zaporozhets2016-10-161-0/+6
|/ | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Merge branch 'atom-routes' into 'master' Robert Speicher2016-10-111-1/+1
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow browsing branches that end with '.atom' ## What does this MR do? 1. Simplify the regex capture in the routing for the CommitsController to not exclude the '.atom' suffix. That's a perfectly valid git branch name, so we shouldn't blow up if we get it. 2. Because Rails now can't automatically detect the request format, add some code to do so in `ExtractPath` when there is no path. This means that, given branches 'foo' and 'foo.atom', the Atom feed for the former is unroutable. To fix this: don't do that! Give the branches different names! ## Why was this MR needed? Creating a branch or tag name ending in '.atom' would cause some 500s on that repo. ## What are the relevant issue numbers? Closes #21955. Related to !5994. See merge request !6750
| * Allow browsing branches that end with '.atom'Sean McGivern2016-10-111-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | We need to do two things to support this: 1. Simplify the regex capture in the routing for the CommitsController to not exclude the '.atom' suffix. That's a perfectly valid git branch name, so we shouldn't blow up if we get it. 2. Because Rails now can't automatically detect the request format, add some code to do so in `ExtractPath` when there is no path. This means that, given branches 'foo' and 'foo.atom', the Atom feed for the former is unroutable. To fix this: don't do that! Give the branches different names!
* | Catch any undefined API routing and return 400 Bad RequestDmitriy Zaporozhets2016-10-101-1/+3
|/ | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Update user routing spec after constrainer logic changeddz-root-urlDmitriy Zaporozhets2016-10-061-1/+1
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Change user & group landing page routing from /u/:name & /groups/:name to /:nameDmitriy Zaporozhets2016-10-061-1/+3
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Fix markdown help referencesClement Ho2016-08-261-2/+2
|
* Support integration with Koding (online IDE)Gokmen Goksel2016-08-191-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Koding: #index: landing page for Koding integration If enabled it will provide a link to open remote Koding instance url for now we are also providing the sneak preview video for how integration works in detail. Repository: check whether .koding.yml file exists on repository Projects: landing page: show Run in IDE (Koding) button if repo has stack file Projects: MR: show Run in IDE Koding button if repo has stack file on active branch ProjectHelpers: add_koding_stack: stack generator for provided project With this helper we will auto-generate the required stack template for a given project. For the feature we can request this base template from the running Koding instance on integration. Currently this will provide users to create a t2.nano instance on aws and it'll automatically configures the instance for basic requirements. Projects: empty state and landing page provide shortcuts to create stack projects_helper: use branch on checkout and provide an entry point This ${var.koding_queryString_branch} will be replaced with the branch provided in query string which will allow us to use same stack template for different branches of the same repository. ref: https://github.com/koding/koding/pull/8597/commits/b8c0e43c4c24bf132670aa8a3cfb0d634acfd09b projects_helper: provide sha info in query string to use existing vms With this change we'll be able to query existing vms on Koding side based on the commit id that they've created. ref: https://github.com/koding/koding/pull/8597/commits/1d630fadf31963fa6ccd3bed92e526761a30a343 Integration: Docs: Koding documentation added Disable /koding route if integration is disabled Use application settings to enable Koding Projects_helper: better indentation with strip_heredoc usage Projects_helper: return koding_url as is if there is no project provided current_settings: set koding_enabled: false by default Koding_Controller: to render not_found once integration is disabled Dashboard_specs: update spec for Koding enabled case Projects_Helper: make repo dynamic ref: https://github.com/koding/koding/pull/8597/commits/4d615242f45aaea4c4986be84ecc612b0bb1514c Updated documentation to have right format
* Rename `markdown_preview` routes to `preview_markdown`Christopher Bartz2016-08-111-4/+4
|
* Add failing test for #20462winniehell2016-08-021-1/+4
|
* Clean up unused routesJosef Strzibny2016-07-263-17/+0
|
* Fix help page paths to make sure shortcuts and the UI help page work.fix-help-pathsConnor Shea2016-07-191-5/+2
| | | | | | Add a test to make sure the help page UI path doesn't break in the future. Fix #19972 and #19889.
* Fix failing tests.Connor Shea2016-07-111-13/+9
|
* Enable Style/EmptyLines cop, remove redundant onesrubocop/enable-cops-for-empty-linesGrzegorz Bizon2016-07-013-3/+0
|
* Merge branch 'health-check-route'Douwe Maan2016-05-122-0/+17
|\ | | | | | | | | # Conflicts: # db/schema.rb
| * Add tests for the health check featureDJ Mountney2016-05-102-0/+17
| |
* | Add specs for user routing and update spec for user controllerLong Nguyen2016-05-081-0/+37
| |
* | user routings refactorLong Nguyen2016-05-081-0/+4
| |
* | Remove unused code, update spec, and update changelogLong Nguyen2016-05-051-8/+0
|/
* Remove `Profile::KeysController#new` actionRobert Speicher2016-03-031-5/+0
|
* Increase the minimum length for commit SHA matching to 7rs-issue-12706Robert Speicher2016-01-291-4/+4
| | | | | | This is the git default and will help to prevent false positive matches. Closes #12706
* Fix broken specs. #2406Rubén Dávila2016-01-201-2/+2
|
* Merge branch 'issue_3076' into 'master' Drew Blessing2016-01-081-0/+1
|\ | | | | | | | | | | | | | | | | Fix problem with projects ending with .keys #3076 Closes #3076 Move route `:username.keys` below project's routes. This allow project's to handle names ending with `.keys` See merge request !1883
| * Fix problem with projects ending with .keys #3076Jose Corcuera2016-01-061-0/+1
| |
* | Merge branch 'master' of github.com:gitlabhq/gitlabhqJacob Vosmaer2016-01-071-0/+12
|/
* Move partial to right place and fix tests.Douwe Maan2015-09-081-3/+3
|
* Revert "Merge branch 'revert-satellites' into 'master' "Dmitriy Zaporozhets2015-08-111-7/+7
| | | | | This reverts commit 5daf44b7c86e0e2641a902b1da8b01d91fa3dbfa, reversing changes made to 2f706fbd231cabe7a76a5d17ac44285aaaf8592c.
* Revert "Merge branch 'drop-satellites'"Dmitriy Zaporozhets2015-08-111-7/+7
| | | | | | | This reverts commit 957e849f41d96fa9778fcdd06792d2f0274b29ab, reversing changes made to 6b9dbe9f5a175a8162abf296367f561bab3eea1a. Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Improve tests to use real branches in merge requestsDmitriy Zaporozhets2015-08-041-7/+7
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Audit log for user authenticationValery Sizov2015-07-061-2/+2
|
* Fix Style/TrailingBlankLines cop violationsRobert Speicher2015-06-222-2/+0
|