Skip to content
This repository has been archived by the owner on Feb 2, 2022. It is now read-only.

Лучше использовать декораторы nestjs, там где это возможно #48

Closed
grigori-gru opened this issue Sep 24, 2019 · 2 comments
Labels
backend Improvements or additions to documentation

Comments

@grigori-gru
Copy link
Contributor

Тут хотел бы обсудить, это моё скорее мнение, если есть возможность, то лучше использовать декораторы. На примере redirect, смысл его использовать, если можно поставить @Redirect('/auth/sign_in') сразу? Это выразительней, как мне кажется, не нужно res дергать лишний раз, плюс ответственность не размывается. Я его у себя стараюсь уж совсем в крайних случаях вытаскивать. На тот же flash можно декоратор сделать, будет хорошим стартовым issue)

@jougene
Copy link
Contributor

jougene commented Sep 24, 2019

Ждем пр.

@grigori-gru
Copy link
Contributor Author

Это все просто топ для первого pr, там чисто декоративные изменения

@jougene jougene added the backend Improvements or additions to documentation label Oct 4, 2019
@jougene jougene closed this as completed Nov 4, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backend Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants