| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
|
|
|
| |
To reduce the number of abstract methods, use polymorphic_paths for
collection routes. Unfortunately for member routes, polymorphic_paths
does not support namespaced classes, so still continue to override
methods.
|
| |
|
|
|
|
| |
We introduced hidden_clusterable_fields in this MR but it is no longer
needed as we have gone back to nested resource routes which should
provide the params such as `project_id`.
|
| |
|
|
| |
Also add missing interface in abstract class.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
Use ClustersController as base while having Projects::ClustersController
to inform what `clusterable` is. Thanks @ayufan for the great suggestion
!
- View changes to work with new approach
- Fix javascript for new approach
- Fix feature specs for new approach
- Fix QA
|
| |
|
|
|
|
|
|
|
|
| |
When we unhooked ClustersController from
Project::ApplicationsController, we missed an EE override to
handle_not_found_or_authorized.
Rather than carry on with override RoutingActions, make a specific proc
for Project that we override in EE instead. Use that proc in both
Clusters::BaseController and Project::ApplicationsController.
|
| |
|
|
|
|
|
| |
- Move show path for cluster to ClusterPresenter
- Create ClusterablePresenter to encapsulate logic. Consolidates
scattered methods from BaseController and ClustersHelper into an object.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
To the extent possible swap out `project` with `clusterable`
- Abstract paths for showing cluster or clusters. This will allow us to
swap in alternative paths for group level cluster
- Push :project_id and :namespace_id params from the URL to the POST
body.
- Create a nice helper for to generate links for the destroy
action
For some reason, spec :project_id and :namespace_id param are not going
through `to_param` for a JSON format. Manually call `to_param` to fix
specs.
- Move :layout to BaseController
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In preparation so that we can create both cluster attached to project
and cluster attached to group.
- Move ClustersController to top level
- Move Clusters::ApplicationsController to top-level too
- Creates a Clusters::BaseController to share common functions
- Do not rely on @project ivar. Anything could set the ivar.
- Fix Vue page components due to new data-page value
Because of the controller change we have gone from
`projects:clusters:new` to `clusters:new`, so we need to update the file
location of the page components. There is somewhere a function that will
convert data-page to a file location.
On that note, projects/clusters/gcp/new/, translate to
Projects::Clusters::Gcp#new doesn't exist so replace that with
clusters/create_gcp/ and clusters/create_user/
|
| |\
| |
| |
| |
| | |
Fix problem with spec that showed up in EE merge
See merge request gitlab-org/gitlab-ce!22675
|
| | | |
|
| |\ \
| |/
|/|
| |
| |
| |
| | |
into 'master'
docs: Add deprecation notice for renamed licensed feature
See merge request gitlab-org/gitlab-ce!22575
|
| | | |
|
| |\ \
| | |
| | |
| | |
| | | |
Added a note about machine types
See merge request gitlab-org/gitlab-ce!22572
|
| |/ / |
|
| |\ \
| | |
| | |
| | |
| | | |
Update gitlab-ui to 1.9.0
See merge request gitlab-org/gitlab-ce!22632
|
| |/ / |
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Remove mousetrap-rails gem
Closes #53270
See merge request gitlab-org/gitlab-ce!22647
|
| | | |
| | |
| | |
| | | |
Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com>
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Make new merge request URL more friendly when pushing code
Closes #53012
See merge request gitlab-org/gitlab-ce!22526
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Add e2e test to push using a private token
See merge request gitlab-org/gitlab-ce!22628
|
| | | | | |
| | | | |
| | | | |
| | | | | |
Tests pushing over HTTP(S) using a personal access token
|
| |\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Update moment to 2.22.2
Closes #53273
See merge request gitlab-org/gitlab-ce!22648
|
| | | |_|/ /
| |/| | |
| | | | |
| | | | | |
Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com>
|
| |\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Update GitLab Pages to v1.3.0
See merge request gitlab-org/gitlab-ce!22666
|
| | | | | | | |
|
| |\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Replace deprecated uniq on a Relation with distinct
See merge request gitlab-org/gitlab-ce!22625
|
| | | | | | | | |
|
| |\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Rename 'sast_container' licensed feature
See merge request gitlab-org/gitlab-ce!22662
|
| | | | | | | | | |
|
| |\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Upgrade whitequark/parser to 2.5.3.0
See merge request gitlab-org/gitlab-ce!22652
|
| | | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
This squelches the following warning:
warning: parser/current is loading parser/ruby24, which recognizes
warning: 2.4.4-compliant syntax, but you are running 2.4.5.
warning: please see https://github.com/whitequark/parser#compatibility-with-ruby-mri.
|
| |\ \ \ \ \ \ \ \ \
| |_|_|_|_|_|/ / /
|/| | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Delete job_spec.js
See merge request gitlab-org/gitlab-ce!22664
|
| | | |_|_|/ / / /
| |/| | | | | | |
|
| |\ \ \ \ \ \ \ \
| |/ / / / / / /
|/| | | | | | |
| | | | | | | |
| | | | | | | | |
Truncate milestone title on collapsed sidebar
See merge request gitlab-org/gitlab-ce!22624
|
| | | | | | | | | |
|
| |\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Fix broken "Show whitespace changes" button on MR "Changes" tab
Closes #52122
See merge request gitlab-org/gitlab-ce!22539
|
| |/ / / / / / / / |
|
| |\ \ \ \ \ \ \ \
| |_|_|/ / / / /
|/| | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Remove all tables from Pipeline page when there is no job for the Pipeline
Closes #45669
See merge request gitlab-org/gitlab-ce!18540
|
| |/ / / / / / / |
|
| |\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
'master'
Extract ee specific lines for issues and mr controllers
See merge request gitlab-org/gitlab-ce!22569
|
| | | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Move access checks to their own method so they can be overridden, and
port an EE-only method to exist in CE too, with an EE-specific override.
|
| | | | | | | | | |
|
| |\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Fix open-ended params for api_json.log
Closes #53155
See merge request gitlab-org/gitlab-ce!22623
|
| |/ / / / / / / / |
|
| |\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
adds warning against Postgres across NFS
See merge request gitlab-org/gitlab-ce!22278
|
| |/ / / / / / / / |
|
