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

jQuery Core 1.9 Upgrade Guide #113

Closed
5 tasks
felixheidecke opened this issue Jul 18, 2018 · 6 comments
Closed
5 tasks

jQuery Core 1.9 Upgrade Guide #113

felixheidecke opened this issue Jul 18, 2018 · 6 comments
Assignees
Milestone

Comments

@felixheidecke
Copy link

jQuery Core 1.9 Upgrade Guide

Changes of Note in jQuery 1.9
See the full upgrade guide at jquery.com

Removed

Minor

.data()

  • /js/impersonate.js

.parents()

  • /js/impersonate.js

.attr()

  • /js/settings-admin.js

.insertBefore()

  • /js/impersonate_logout.js

.insertAfter()

  • /js/impersonate.js
@sharidas
Copy link
Contributor

Created PR #114, as its required to show impersonate icon.
Also applied the changes mentioned at owncloud/core#31739 (comment).
Below are the tests conducted:

  • login as admin user
  • created 3 users user1, user2, user3
  • enabled impersonate
  • Navigated to user management page
  • Impersonate icon appeared for user1, user2 and user3.
  • created a group group1 and added users user1 and user2 to the group. user1 and user3 are the group admin for group1
  • Whitelisted the group group1 in the Settings -> User Authentication page.
  • Now tried to impersonate user1 successfully. Was able to see impersonate icon only for user2.
  • Now tried to impersonate user2. Didn't allowed further impersonation. Expected behavior.
  • Logged out and reached back to admin.
  • Logged in as user3 and was able to see impersonate icon for user1 and user2.

@sharidas
Copy link
Contributor

sharidas commented Jul 25, 2018

This works as expected. Provided we would have to merge the change:

and will have to bring change as mentioned in owncloud/core#31739 (comment)

@PVince81
Copy link
Contributor

the change was merged, anything else ?

@PVince81
Copy link
Contributor

regarding owncloud/core#31739 (comment), please test against stable10 instead if possible

@PVince81
Copy link
Contributor

PVince81 commented Aug 6, 2018

@sharidas please tick the checkboxes accordingly for things you verified / fixed

@sharidas
Copy link
Contributor

sharidas commented Aug 8, 2018

Yes other than owncloud/core#31739 (comment), from the impersonate apps perspective nothing blocks.

Closing this ticket as #114 is merged.

@sharidas sharidas closed this as completed Aug 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants