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

Breaks user admin in django-suit #48

Closed
paketep opened this issue Mar 15, 2016 · 5 comments
Closed

Breaks user admin in django-suit #48

paketep opened this issue Mar 15, 2016 · 5 comments
Assignees

Comments

@paketep
Copy link

paketep commented Mar 15, 2016

Hi!.

Great work with the plugin, it's really useful.

However, it looks like it doesn't like to work with django-suit. "Login as" works ok in the changelist, but it breaks completely the user form in the admin.

Thanks!

@bashu
Copy link
Contributor

bashu commented Mar 23, 2016

@paketep please try pip install -U django-su

@paketep
Copy link
Author

paketep commented Mar 23, 2016

Thanks, bashu.

I'm out of the office for Easter, but I'll test your fix as soon as I get back on Tuesday.

Have a nice week!

@paketep
Copy link
Author

paketep commented Mar 30, 2016

Hi, bashu!

Sorry it took me a bit more to test your changes.

User admin page now looks great and works great as well. In the beginning it was deleting my session, but I think it was because I didn't remove 0.5.0 before installing 0.5.1.

However, now selecting a user in the /su/login page gives me a CSRF error. It was working ok with 0.5.0 (which I reinstalled to be sure).

Also, I don't know why, the "Login as another user" button in that page looks different because the anchor tag is inside a List Item. I don't know if there's a way to fix that in the plug-in or if I should retouch the template. Here's a screenshot:

051a

Since you closed he issue, I can't reopen it. Hope you're still subscribed.

Thank you!

@bashu
Copy link
Contributor

bashu commented Apr 20, 2016

see darklow/django-suit#499, in a meantime you can override su/login_link.html template

@bashu bashu reopened this Apr 20, 2016
@paketep
Copy link
Author

paketep commented Apr 28, 2016

Great, I put the template in my app while I wait for suit's next version, and now it looks perfect.

Thank you!

@paketep paketep closed this as completed Apr 28, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants