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

Customer order param is not working in notification templates. #129

Open
yecli opened this issue Jan 22, 2020 · 0 comments
Open

Customer order param is not working in notification templates. #129

yecli opened this issue Jan 22, 2020 · 0 comments

Comments

@yecli
Copy link
Contributor

yecli commented Jan 22, 2020

Steps:

  1. Open platform manager, Notifications tab;
  2. Find Order paid notification, create it if it does not exist;
  3. Select template, press Send;
  4. Enter valid sender and recipient;
  5. Enter something (e.g. order number) into the customer order field;
  6. Press Send

Actual result:
500 error

Expected result:
No error. We should either have the ability to enter the data and process with it, or not see this field at all (or see it disabled).

Details:
Now this field is used as CustomerOrder object, and that causes the error - as it is just a string, in fact. We could mark such properties as "not manually set" by attribute (as it is set in the code), or not show it at all based on "complex" type.
Or we could have an ability to pass test object, e.g. using Json and restore full object on the server side.

image

@yecli yecli changed the title Customer order param i snot working in notification templates. Customer order param is not working in notification templates. Jan 23, 2020
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

1 participant