Conversation
…tion in textbox config form
AlexShmak
force-pushed
the
feature/textbox
branch
from
25c23d7 to
4955978
Compare
| .side-menu-header { | ||
| position: relative; | ||
| text-align: center; | ||
| padding: 6px 28px; | ||
| } | ||
|
|
||
| .side-menu-header span { | ||
| font-size: inherit; | ||
| user-select: none; | ||
| } | ||
|
|
||
| .side-menu-arrow { | ||
| position: absolute; | ||
| top: 50%; | ||
| transform: translateY(-50%); | ||
| cursor: pointer; | ||
| font-size: 1.2rem; | ||
| color: #6c757d; | ||
| opacity: 0.6; | ||
| } | ||
|
|
||
| .side-menu-arrow.left { | ||
| left: 6px; | ||
| } | ||
|
|
||
| .side-menu-arrow.right { | ||
| right: 6px; | ||
| } | ||
|
|
||
| .side-menu-arrow:hover { | ||
| color: #000; | ||
| opacity: 1; | ||
| } |
There was a problem hiding this comment.
С новым side menu мне кажется очень крутая идея 👍 Учитывая что скорее всего ещё новые устройства появлятья будут
back/src/network_topology.py
Outdated
| node_id: str = node.data.id # network device name(label) | ||
|
|
||
| if node_type == "textbox": | ||
| return |
There was a problem hiding this comment.
Может быть стоит сделать фильтрацию просто по всем неизвестным типам, а не конкретно по текстбоксам? А вообще, хорошо было бы, чтобы бэкенд ничего не знал о текстбоксах
There was a problem hiding this comment.
Окей, посмотрю как лучше на бэке отфильтровать подобные элементы
back/src/network_schema.py
Outdated
| """ | ||
|
|
||
| config: NodeConfig | ||
| config: NodeConfig | TextboxNodeConfig |
There was a problem hiding this comment.
TextboxNodeConfig же унаследован от NodeConfig. Кажется явно не то что-то с абстракциями
There was a problem hiding this comment.
Справедливо, изначально так и сделал, но почему-то сломалась десериализация. В целом если текстбокс будет все равно отфильтровываться (в будущем исправлении), то он и не дойдет до этого момента
back/src/network_schema.py
Outdated
| default_gw: str = "" | ||
|
|
||
|
|
||
| @dataclass |
There was a problem hiding this comment.
Вообще, под Node на бэкенде раньше понималось network node, т.е. какое-то сетевое устройство. Кажется, как будто отдельный TextboxNodeConfig немного нарушает эту идею
- не очень понимаю, textbox это же чисто элемент фронтенда, зачем он в эмуляторе?
There was a problem hiding this comment.
На бэке перебираются все ноды и десериализуются. Изначально подумал, что node на бэке -- cytoscape node.
В таком случае можно отфильтровать текстбоксы и не обрабатывать (предыдущее обсуждение), согласен, сделаю
There was a problem hiding this comment.
На бэке перебираются все ноды и десериализуются. Изначально подумал, что
nodeна бэке --cytoscape node
Если бы в проекте бэкенд и эмулятор были корректно разделены, то путаницы наверное бы не было
|
|
||
| # Режим работы: dev (локальный PostgreSQL) или prod (Yandex Cloud PostgreSQL) | ||
| MODE=prod No newline at end of file | ||
| MODE=dev No newline at end of file |
There was a problem hiding this comment.
😠😠😠😠😠😠😠😠😠😠😠 надо этот файл в .gitignore добавить
There was a problem hiding this comment.
По-хорошему он там должен был быть уже давно (изначально), текущие .env с примерной конфигурацией стоит оставить с MODE=dev и переименовать во что-то, вроде example.env. Занести в .gitignore и не трогать продовый .env
d-zaytsev
left a comment
d-zaytsev
left a comment
There was a problem hiding this comment.
Несколько моментов на подумать написал
AlexShmak
force-pushed
the
feature/textbox
branch
from
50c0fae to
6995eae
Compare
…unknown node types
AlexShmak
force-pushed
the
feature/textbox
branch
from
6995eae to
3feb67b
Compare
2 participants
Add textbox tool for creating comments on canvas: