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

Frontend area of payment info block shows too much #14

Closed
maderlock opened this issue Oct 13, 2017 · 5 comments
Closed

Frontend area of payment info block shows too much #14

maderlock opened this issue Oct 13, 2017 · 5 comments

Comments

@maderlock
Copy link

The info block for a payment is rendered in the customer account area for orders, as well as in the order confirmation email.

I believe that it should only show relevant information for customers, but at present it's showing all transaction data that the admin sees. Here's an example from an order email:

screenshot 2017-10-13 15 20 41

The area is set on the block before it's rendered, so you should know if it's frontend or admin. If the former then I think you could just be showing the method and none of the extra info. Perhaps not output anything for frontend?

@david-fiaty-cko
Copy link
Contributor

issue fixed in new version 1.0.8

@maderlock
Copy link
Author

This seems to have fixed the customer order display in the account area, but I see the same issue with 1.0.8 in order emails. Note that static content was re-compiled as was DI generated files.

@pradeep51
Copy link

D:\xampp2\htdocs\Magento2> rm -rf var/cache var/generation/ var/di
'rm' is not recognized as an internal or external command,
operable program or batch file.

@moneef moneef mentioned this issue Oct 31, 2017
@maderlock
Copy link
Author

@pradeep51 That is a unix command and you are using Windows. Please would you mind asking questions like this on somewhere like stack overflow as this is not directly relevant to the checkout.com extension?

@david-fiaty-cko
Copy link
Contributor

@maderlock the payment info remaining in the new order email has been removed in the latest version ( v1.0.11).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants