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.

Sign up for GitHub

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

Jump to bottom

Asynchronous User List Loading, Fixed Issues 2005, 2014, 2021 #21

Open
wants to merge 21 commits into
base: grouper-integration
Choose a base branch
from
Open

Asynchronous User List Loading, Fixed Issues 2005, 2014, 2021 #21

wants to merge 21 commits into from

Conversation

jeffkinnison
Copy link
Contributor

@jeffkinnison jeffkinnison commented Aug 15, 2016 •
edited
Loading

Updates:

  • Requirements in the "Data Sharing Requirement" document addressed
  • Client-side sharing utilities now load user lists via AJAX to reduce loading time
  • Routes added to ExperimentController and ProjectController that return a JSON list of users
  • Experiment Browse page now only loads experiments that the user has Grouper permissions for
  • Default Project owner now has full permissions for the Default Project

@jeffkinnison
Switching to load users on an AJAX call
3c07153
@jeffkinnison
updated controllers to only load shared users, added functions to ret…
4f5f026 
…rieve user lists as JSON
@jeffkinnison
changed AJAX call to POST and added resourceId field
84f160a
@jeffkinnison
updated views to incorporate new user loading scheme
b742a23
@jeffkinnison
updated views to incorporate new user loading scheme
878df05
@jeffkinnison
added user list routes to project and experiments
7206631
@jeffkinnison
user lists load asynchronously
bb852b7
@jeffkinnison
fixed issue 2005, 2014, 2021
98b6ca3
@jeffkinnison
minor fixes
7b21101
@jeffkinnison
Project summary page now only load experiments that the user may access
dcf8e47
@jeffkinnison
Default Project owner granted read and write permissions at creation
39f9b23
@jeffkinnison
project owner always granted sharing when permissions are set (to ens…
fe50621 
…ure that even if something goes wrong on the client end, the owner can still access their project)
@jeffkinnison
Removed ability for users with read only permissions to access projec…
cb57e29 
…t edit pages
@jeffkinnison
changes to ensure project owner cannot have permissions changed by ot…
9f0ee4e 
…her users
@jeffkinnison
redirect read-inly users from edit view to summary view
ca0105c
@jeffkinnison
minor logic change
a5dbbaa
@jeffkinnison
Removed link to edit page to read-only users
2fe80e7
@jeffkinnison
Removed edit links from browse
ebdc163
@jeffkinnison
Added owner thumbnail, post-completion sharing, and verified that the…
face8c5 
… changes work
@jeffkinnison
fixed experiment ownership nomenclature issue
81628aa
@jeffkinnison
Fixed create-complete screen
9ff7296
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jeffkinnison