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

Deleting a user with orders throws Missing Template exception #3126

Closed
brchristian opened this issue Feb 27, 2019 · 3 comments · Fixed by #4415
Closed

Deleting a user with orders throws Missing Template exception #3126

brchristian opened this issue Feb 27, 2019 · 3 comments · Fixed by #4415
Labels
type:bug Error, flaw or fault

Comments

@brchristian
Copy link
Contributor

When attempting to delete a user from the backend who has orders, the deletion will fail and a Missing Template error screen will be shown. (Screenshot below.)

Solidus Version:
2.8.1

To Reproduce
Create new 2.8.1 app.
Complete an order with a user.
Attempt to delete user from backend.

Current behavior
Missing Template exception will be seen.

Expected behavior
I would expect either to see an explanation to the effect of Cannot delete user with orders, or a dialogue This user has orders. Continue?, or a successful deletion.

Screenshots
screen shot 2019-02-27 at 12 56 19 pm

Desktop (please complete the following information):

  • OS: MacOS Mojave
  • Browser Safari
  • Version 12.0.3
@kennyadsl kennyadsl added the type:bug Error, flaw or fault label Feb 28, 2019
@jacobherrington
Copy link
Contributor

jacobherrington commented Mar 13, 2019 •
edited
Loading

This still exists in master. I'll try to make a fix today. I think @aitbw is on it.

This was referenced Mar 13, 2019
Closed
Merged
@jacobherrington
Copy link
Contributor

Just as an update, the change in #3139 does not close this issue. It is only a temporary fix.

@tvdeyen
Copy link
Member

tvdeyen commented Jul 26, 2022

Fixed with #4415

@tvdeyen tvdeyen closed this as completed Jul 26, 2022
@tvdeyen tvdeyen linked a pull request Jul 26, 2022 that will close this issue
Fix delete response in admin users controller #4415
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type:bug Error, flaw or fault
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix delete response in admin users controller tvdeyen/solidus
4 participants
@tvdeyen @kennyadsl @brchristian @jacobherrington