Commit graph

10 commits

Author SHA1 Message Date
James Fargher
f07d445b0c Remove duplicate clusterable presenter method 2019-05-07 08:49:27 +12:00
Peter Leitzen
c8a530a319 Show health graphs on group-level
Tweak cluster helper and refactor its specs.
2019-05-06 16:24:14 +00:00
João Cunha
1186a6fd54 Sends update route to the client
- extends presenters to include update endpoint path
- sends path to the client on clusters clusters show view.
2019-03-05 08:32:51 +02:00
Thong Kuah
ecb4a29fc0 Reverse view override as EE has an override too
Use presenters instead otherwise we will have x2 overides in EE
2018-11-08 23:28:30 +13:00
Thong Kuah
b019224ff2 Refactor to use presenter
Refacotr project or group specific `if` logic to be contained within
each presenter instead.

Also add :override to ensure super method exists.
2018-11-08 23:28:30 +13:00
Mike Greiling
83b8321aea Rename is_project? to project? 2018-11-08 23:28:30 +13:00
Mike Greiling
1dba5dbc3a Update empty state for group clusters index 2018-11-08 23:28:30 +13:00
Thong Kuah
fec21f5542 Use polymorphic_paths where we can
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.
2018-11-02 10:41:53 +13:00
Thong Kuah
1a1fdf8efe Resolve controller sharing concern
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
2018-11-01 19:37:33 +13:00
Thong Kuah
1163b23539 Move view and path concerns to presenters
- Move show path for cluster to ClusterPresenter

- Create ClusterablePresenter to encapsulate logic. Consolidates
scattered methods from BaseController and ClustersHelper into an object.
2018-11-01 19:37:32 +13:00