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

Deleting a customer note throws an error #435

Closed
listerr opened this issue Jul 11, 2018 · 3 comments
Closed

Deleting a customer note throws an error #435

listerr opened this issue Jul 11, 2018 · 3 comments

Comments

@listerr
Copy link
Contributor

listerr commented Jul 11, 2018

ISSUE TYPE

Bug Report

VERSION
4.8.0
SUMMARY

Deleting a customer note throws an error.

STEPS TO REPRODUCE

Access customer page.
Click notes tab
Click delete on a note in the list.

EXPECTED RESULTS

Note should be deleted and disappear from the list.

ACTUAL RESULTS

Pop up: Error running ajax query for https://servername/api/v4/customer-note/delete/101

  • When refreshing the page, The note appears to have then been deleted anyway.
IMPORTANCE

Minor

@yannrobin
Copy link
Contributor

Hello,

Does it work when you add a new note ? Do you receive the email of notification ?

IXP manager send notification emails when you add/delete/modify a customer note. It might come from that.

Thanks

@listerr
Copy link
Contributor Author

listerr commented Jul 12, 2018

When adding a new note, I get Error! Could not save your note.

It does add it though if I close that error and go back to it.

@listerr
Copy link
Contributor Author

listerr commented Jul 12, 2018

Okay - just to update: There was a superadmin user with a blank email address. I've since added an address to this profile and opted the user out of note notifications.

This seems to have solved the issue.

yannrobin added a commit that referenced this issue Jul 12, 2018
yannrobin added a commit that referenced this issue Jul 12, 2018
yannrobin added a commit that referenced this issue Jul 12, 2018
@barryo barryo closed this as completed in 5183753 Jul 13, 2018
barryo pushed a commit that referenced this issue Jul 13, 2018
… if customer has no email (#436)

* [IM] - Customer note notification : Dont send notification email if customer has no email - fixes #435

* [IM] - Customer note notification : Dont send notification email if customer has no email - fixes #435

* [IM] - Customer note notification : Dont send notification email if customer has no email - fixes #435
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