summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRobert Speicher <robert@gitlab.com>2017-10-09 13:50:14 +0000
committerRobert Speicher <robert@gitlab.com>2017-10-09 13:50:14 +0000
commitd976941a8241dd0f5cdcd0e5cdb49f597fb4c349 (patch)
tree502483a33d50f8ee2915cbf9da3ba9df98bd70b4
parent5d865bccecfb21854afd5c9b71ba67f007e11f09 (diff)
parent2409acfa7f97fef81dd66640c64e9d0008b85cde (diff)
downloadgitlab-ce-d976941a8241dd0f5cdcd0e5cdb49f597fb4c349.tar.gz
Merge branch 'group-sort-dropdown-blank' into 'master'
Fixed default group sort option Closes #38808 See merge request gitlab-org/gitlab-ce!14728
-rw-r--r--app/controllers/dashboard/groups_controller.rb2
-rw-r--r--changelogs/unreleased/group-sort-dropdown-blank.yml5
-rw-r--r--spec/features/dashboard/group_spec.rb6
3 files changed, 12 insertions, 1 deletions
diff --git a/app/controllers/dashboard/groups_controller.rb b/app/controllers/dashboard/groups_controller.rb
index 8057a0b455c..7ed18fb481c 100644
--- a/app/controllers/dashboard/groups_controller.rb
+++ b/app/controllers/dashboard/groups_controller.rb
@@ -1,6 +1,6 @@
class Dashboard::GroupsController < Dashboard::ApplicationController
def index
- @sort = params[:sort] || 'id_desc'
+ @sort = params[:sort] || 'created_desc'
@groups =
if params[:parent_id] && Group.supports_nested_groups?
diff --git a/changelogs/unreleased/group-sort-dropdown-blank.yml b/changelogs/unreleased/group-sort-dropdown-blank.yml
new file mode 100644
index 00000000000..dd16892be4d
--- /dev/null
+++ b/changelogs/unreleased/group-sort-dropdown-blank.yml
@@ -0,0 +1,5 @@
+---
+title: Fixed group sort dropdown defaulting to empty
+merge_request:
+author:
+type: fixed
diff --git a/spec/features/dashboard/group_spec.rb b/spec/features/dashboard/group_spec.rb
index 970173c7aaf..1213f8c32eb 100644
--- a/spec/features/dashboard/group_spec.rb
+++ b/spec/features/dashboard/group_spec.rb
@@ -5,6 +5,12 @@ RSpec.describe 'Dashboard Group' do
sign_in(create(:user))
end
+ it 'defaults sort dropdown to last created' do
+ visit dashboard_groups_path
+
+ expect(page).to have_button('Last created')
+ end
+
it 'creates new group', :js do
visit dashboard_groups_path
find('.btn-new').trigger('click')