Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multi Select Request Access: Grant list box has unordered file list, unlike file view on dataset page. #2572

Closed
kcondon opened this issue Sep 22, 2015 · 7 comments
Assignees
Labels
Feature: File Upload & Handling Feature: Permissions Type: Bug a defect UX & UI: Design This issue needs input on the design of the UI and from the product owner

Comments

@kcondon
Copy link
Contributor

kcondon commented Sep 22, 2015

Grant list box has unordered file list, unlike file view on dataset page.

Makes it more difficult to compare to dataset if the ordering is different. Same with file order on view granted file list, unordered.

@kcondon kcondon added UX & UI: Design This issue needs input on the design of the UI and from the product owner Priority: Medium Type: Bug a defect labels Sep 22, 2015
@kcondon kcondon added this to the 4.2 milestone Sep 22, 2015
@scolapasta scolapasta modified the milestones: In Review, 4.2 Sep 23, 2015
@sekmiller sekmiller removed their assignment Jan 28, 2016
@scolapasta scolapasta removed this from the Not Assigned to a Release milestone Jan 28, 2016
@mheppler
Copy link
Contributor

mheppler commented Feb 8, 2016

The table order is determined by the attribute rowKey="#{roleAssignment.id}".

mheppler added a commit that referenced this issue Feb 8, 2016
…Access popups on the File Permissions pg. [ref #2572]
@mheppler
Copy link
Contributor

mheppler commented Feb 8, 2016

Fixed sort order of the file table on the Grant FIle Access and File Access popups on the File Permissions pg. Also added icons to action buttons to better distinguish Grant from Reject.

This code was pushed to the "2572-permissions-ux" branch.

@mheppler
Copy link
Contributor

mheppler commented Feb 8, 2016

Sorry, taking this back. Don't seem to have this solved on all three dataTables on the File Permissions pg. The dataTable (Line 236) in the "Grant Access to Users/Groups" popup is still looking random by rowKey="#{file.id}" and not sorted by sortBy="#{file.displayName}".

@mheppler
Copy link
Contributor

mheppler commented Feb 9, 2016

The "Grant File Access" popup seems to be the only place that the sortBy attribute is working for. (See attached.)

Sort order needs to be fixed in the following file or users/groups dataTables across the various permissions workflows.

Dataverse/Dataset Permissions

  • Users/Groups panel > Users/Groups dataTable

File Permissions

  • Users/Groups panel > Requests dataTable
  • Users/Groups panel > Grant File Access popup > File Name dataTable
  • Users/Groups panel > Users/Groups dataTable
  • Users/Groups panel > File Access popup > File Name dataTable
  • Users/Groups panel > Grant File Access popup > File Name dataTable
  • Restricted Files panel > Restricted Files dataTable
  • Restricted Files panel > File Access popup > User/Group Name dataTable
  • Restricted Files panel > Grant File Access popup > File Name dataTable

screen shot 2016-08-05 at 12 53 19 pm

screen shot 2016-08-05 at 12 54 55 pm

@djbrooke
Copy link
Contributor

djbrooke commented Aug 1, 2016

The fix for #2657 is in this branch.

@mheppler
Copy link
Contributor

mheppler commented Aug 5, 2016

Updated my comment above to make the checklist items clearer as to which dataTable was broken and in which popup from which page.

@djbrooke djbrooke added this to the 4.6 - File Replace milestone Aug 5, 2016
sekmiller added a commit that referenced this issue Aug 12, 2016
@djbrooke djbrooke modified the milestones: 4.5.1 - IP Groups fixes, 4.6 - File Replace Sep 1, 2016
@djbrooke djbrooke assigned scolapasta and unassigned sekmiller Sep 8, 2016
@sekmiller sekmiller mentioned this issue Sep 12, 2016
7 tasks
sekmiller added a commit that referenced this issue Sep 13, 2016
@scolapasta scolapasta removed their assignment Sep 13, 2016
@kcondon kcondon closed this as completed Sep 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature: File Upload & Handling Feature: Permissions Type: Bug a defect UX & UI: Design This issue needs input on the design of the UI and from the product owner
Projects
None yet
Development

No branches or pull requests

5 participants