diff options
author | Phil Hughes <me@iamphill.com> | 2017-10-06 14:42:01 +0100 |
---|---|---|
committer | Phil Hughes <me@iamphill.com> | 2017-10-06 16:07:09 +0100 |
commit | 0f366c74131339cb45c8943437fe5b3e68721c75 (patch) | |
tree | 247be2df30143c3ba2a3beb11bf7c6aedc7c4142 | |
parent | 050548032475458a70005ed3f7ff74211084a423 (diff) | |
download | gitlab-ce-group-sort-dropdown-blank.tar.gz |
Fixed default group sort optiongroup-sort-dropdown-blank
Closes #38808
-rw-r--r-- | app/controllers/dashboard/groups_controller.rb | 2 | ||||
-rw-r--r-- | changelogs/unreleased/group-sort-dropdown-blank.yml | 5 | ||||
-rw-r--r-- | spec/features/dashboard/group_spec.rb | 6 |
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 60a16830cdc..3e61fafff69 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: true do visit dashboard_groups_path find('.btn-new').trigger('click') |