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

Добавить возможность прикреплять файл к письму с заказом. #802

Open
ShevArtV opened this issue Feb 9, 2023 · 0 comments

Comments

@ShevArtV
Copy link
Collaborator

ShevArtV commented Feb 9, 2023

Сейчас эту задачу можно решить одним из следующих способов

  1. Переопределить метод - это плохо, так как крайне нежелательно, чтобы сторонние компоненты переопределяли родные методы, а генерацией и прикреплением файлов, по задумке, будут заниматься именно они.
  2. Передавать ссылку на файл в чанк письма - это плохо потому, что файлы надо где-то хранить достаточно продолжительное время, а это требует места на хостинге и это место может внезапно закончится и тогда вообще сайт ляжет.

Я предлагаю добавить 4-й параметр в метод sendEmail attach по умолчанию пустой, в заказ добавить поле attach для хранения пути к файлу, а в методе changeOrderStatus значение из этого поля передавать в метод sendEmail.

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