Repository navigation
Финальный отсчёт - #12
Conversation
| console.log(this.uploadDirectory); | ||
| console.log(this.fieldName); |
There was a problem hiding this comment.
Так как проект готовим к сдаче, давай назовём это production, то логи такие лучше убрать, у этого на самом деле есть причина. Нам надо снижать кол-во логов, особенно console.log, так как это достаточно нагруженная операция, а так же она синхронная, то есть будет блокировать поток, а так же ты тут используешь два отдельных.
Да это не такая сильная проблема, но я бы приучал себя, чтобы логировали только то что правда нужно в проде
| new UploadFileMiddleware( | ||
| 'avatar', | ||
| this.config.get('UPLOAD_FILE_DIRECTORY'), | ||
| ), |
There was a problem hiding this comment.
Ты поменял порядок аргументов и кажется это теперь не правильно работает. В целом чтобы такое избегать зачастую, если хотя бы 2 параметра уже создают interface, чтобы передавать аргументы именовано -> { arg1: val1, arg2: arg2 }
| const comments = await this.commentService.findByOfferId(offerId as string); | ||
|
|
||
| this.ok(res, fillDTO(OfferRDO, comments)); |
There was a problem hiding this comment.
Поправь меня, но тут мы получаем комментарии, но заполняем модель оффера, это так задумано?
| return this.commentModel | ||
| .find({ offerId }) | ||
| .populate('author') | ||
| .limit(50) |
There was a problem hiding this comment.
Давай избавимся от магических цифр, и вынесем это число в констату, это повышает читабельность кода, а так же поддерживать становится легче.
🎓 Финальный отсчёт