An error occurred fetching the project authors.
- 09 Nov, 2018 4 commits
-
-
Thong Kuah authored
Use presenters instead otherwise we will have x2 overides in EE
-
Thong Kuah authored
Refacotr project or group specific `if` logic to be contained within each presenter instead. Also add :override to ensure super method exists.
-
Mike Greiling authored
-
Mike Greiling authored
-
- 01 Nov, 2018 6 commits
-
-
Thong Kuah authored
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.
-
Thong Kuah authored
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.
-
Thong Kuah authored
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
-
Thong Kuah authored
- Move show path for cluster to ClusterPresenter - Create ClusterablePresenter to encapsulate logic. Consolidates scattered methods from BaseController and ClustersHelper into an object.
-
Thong Kuah authored
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
-
Thong Kuah authored
- Move show path for cluster to ClusterPresenter - Create ClusterablePresenter to encapsulate logic. Consolidates scattered methods from BaseController and ClustersHelper into an object.
-