View templatesRemove email "Preview" from BO section "Statutes" (because it's not working)

[ShaiMagal]

Questions	Answers
Branch?	8.1.x
Description?	Check issue #36165
Type?	improvement
Category?	BO
BC breaks?	no
Deprecations?	no
How to test?	Go to any Order State with mail template and check, if there is button "Preview"
UI Tests
Fixed issue or discussion?	Fixes #36165
Related PRs
Sponsor company	https://www.openservis.cz/

[kpodemski]

That's not how we should do things, removing something because it stopped working is not a solution.

[kpodemski]

Isn't a solution to simply allow browsing these files in the .htaccess?

[Hlavtox]

@kpodemski probably a security issue, everyone could browse your templates

[ShaiMagal]

@kpodemski But this is not working for 4 years (#17759)... It's nothing new...

So better option is to leave this still not working and still explaining customers? seriously? :-D

Better option for me is hide this, if this is not working for 4 years already. Customers are complaining. Hiding is not ideal solution, but better than leave this feature still broken....... :/

Unfortunatelly, no :( because, it will allow "attacker" to see content of this mail templates. And this is purpose of .htaccess to hide this from public.

[kpodemski]

Ok. From the code perspective, it's good, from the product point of view, I'd like to see @MatShir opinion on this one, I think it's not the end of the world to temporarily disable this.

[MatShir]

LGTM 👌
thanks @ShaiMagal !