-
Notifications
You must be signed in to change notification settings - Fork 13
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
Переименовать все namespace.ts в types.ts #148
Comments
давно пора)) |
ага и импортировать как * as T, as TS |
|
идея хорошая, но есть момент с моделями: часто возникает потребность в модулях которые имеют собственные модели и чтобы не путать с моделями приложения я не называю их моделями. Если допускать что в рамках приложения у нас разные множества моделей существуют, тогда каждое множество должен быть подчёркнуто именем модуля в котором оно находится, но доменные модели сейчас лежат в shared/types/models поэтому нужно завести каталог для модуля по работе с доменной областью, src/domainCore например. Тогда всё станет органичным |
Ну мне кажется норм, что у тебя есть корневые модели, если модели модуля. Можно, конечно, доменные в корне складывать и называть просто domain, но там же не только модели предметки могут лежать :) Поэтому я предлагаю пока не заморачиваться, сделать модели в src как корневые, и если надо модуль может свои локальные модели тоже объявлять. По пути иерархия и так будет четко прослеживаться |
можно поменять для ясности, также вместо import * as React заменить на import React с флагом esModuleInterop: true |
Не вижу ничего общего с определением https://en.wikipedia.org/wiki/Namespace
Добавляет путаницы
The text was updated successfully, but these errors were encountered: