Size: a a a

NestJS — русскоязычное сообщество

2020 March 14

IK

ILshat Khamitov in NestJS — русскоязычное сообщество
Я так и не сделал рефрешь его через хттп онли надо делать
источник

S

Sviatoslav in NestJS — русскоязычное сообщество
Товарищи комьюнити, есть спорный вопрос, давайте по рассуждаем, написал модуль для nodemailer, как полагается без зависимостей, понятное дело, что к нему практически всем нужен handlebars, вот не знаю как поступить, заставлять всех его скачивать, даже тем кому он не нужен, или воспользоваться старым импортом? Ну вот как пример https://github.com/svtslv/nestjs-mailer/blob/master/src/mailer.features.ts
источник

IK

ILshat Khamitov in NestJS — русскоязычное сообщество
Sviatoslav
Товарищи комьюнити, есть спорный вопрос, давайте по рассуждаем, написал модуль для nodemailer, как полагается без зависимостей, понятное дело, что к нему практически всем нужен handlebars, вот не знаю как поступить, заставлять всех его скачивать, даже тем кому он не нужен, или воспользоваться старым импортом? Ну вот как пример https://github.com/svtslv/nestjs-mailer/blob/master/src/mailer.features.ts
есть уже такой моудль, там чувак заставлял качать пуг, пришлось себе забрать и выпилить его)
источник

S

Sviatoslav in NestJS — русскоязычное сообщество
ILshat Khamitov
есть уже такой моудль, там чувак заставлял качать пуг, пришлось себе забрать и выпилить его)
там еще и ошибки безопасности сыпятся
источник

И

Илья | 😶 in NestJS — русскоязычное сообщество
вынес мейлинг на отдельный сервис (на ts) и по кайфу, полное флексибилити
источник

И

Илья | 😶 in NestJS — русскоязычное сообщество
В идеале бы на нест завязать...
источник

IK

ILshat Khamitov in NestJS — русскоязычное сообщество
делай без ничего, и конфиг дай для рендера, придумай общий механизм и кастом чтобы был
источник

IK

ILshat Khamitov in NestJS — русскоязычное сообщество
птом запили два нпм плагина чтобы через конфиг можно было указывать
источник

S

Sviatoslav in NestJS — русскоязычное сообщество
ILshat Khamitov
делай без ничего, и конфиг дай для рендера, придумай общий механизм и кастом чтобы был
Так в том то и дело, если я сделаю через require, в случаи использования без барса я смогу кинуть ошибку, мол установи, это идет просто как отдельная фишка, так как в закоментированном коде
источник

S

Sviatoslav in NestJS — русскоязычное сообщество
ну в смысле, отдельная функция которую ты можешь импортнуть с пакета а можешь и нет, если импортнул и используешь без барса тогда еррор если не используешь все ок
источник

S

Sviatoslav in NestJS — русскоязычное сообщество
собственно там в ридми пример
источник

IK

ILshat Khamitov in NestJS — русскоязычное сообщество
я против изобретения того что уже есть)
источник

IK

ILshat Khamitov in NestJS — русскоязычное сообщество
так насоздовали всяких вуев и реактов
источник

IK

ILshat Khamitov in NestJS — русскоязычное сообщество
прикинь если каждый в этом чате создаст по плагину для всего, от этого новичкам тока хуже будет
источник

IK

ILshat Khamitov in NestJS — русскоязычное сообщество
нет чтобы всем качать одно
источник

S

Sviatoslav in NestJS — русскоязычное сообщество
Ну так пускай сделает по нормальному, без зависимостей без кучи ненужного барахла, какой вообще смысл оборачивать nodemailer если это можно сделать простой функцией, и плюс там ощибки сыпятся, из за того что он поустанавливал библиотек с 1 лайком
источник

IK

ILshat Khamitov in NestJS — русскоязычное сообщество
пр запили
источник

IK

ILshat Khamitov in NestJS — русскоязычное сообщество
ща вот уже 4 часа у себя в либе пр разгребаю, слил как то и терь баг через пол года)
источник

S

Sviatoslav in NestJS — русскоязычное сообщество
ILshat Khamitov
пр запили
Думаешь он не знает? Он сам эти зависимости добавлял)
источник

IK

ILshat Khamitov in NestJS — русскоязычное сообщество
обоснуй, мне люди обьясняют почему они меняют мой код)
источник