Conversation
| import { Route, Routes } from 'react-router-dom'; | ||
|
|
||
| import { HomePage, LoginPage } from '@/pages'; | ||
| import { BlacklistPage } from '@/pages/blacklist-page/ui/BlacklistPage'; |
There was a problem hiding this comment.
import { HomePage, LoginPage, BlacklistPage } from '@/pages';
There was a problem hiding this comment.
эту страницу можно удалить
| import {RemoveBlacklistWidget, Header} from "@/widgets"; | ||
| import {Table, TableData, Title} from '@mantine/core'; | ||
|
|
||
| const tableData: TableData = { |
There was a problem hiding this comment.
для данных таблицы лучше сделать type используя TypeScript(по модели из бекенда обычно делается), но можно просто пока как в данном примере поля в типе сделать. Сами данные таблицы вынести в config.ts. Тип для вынести в model/types.ts
| @@ -0,0 +1,21 @@ | |||
| import {RemoveBlacklistWidget, Header} from "@/widgets"; | |||
There was a problem hiding this comment.
- Как я понимаю это модальное окно, в мантин уже есть готовое и можно использовать его + встроенные хуки для обработки
- это можно перенести в папку страницы блеклиста, так как он только там используется. Заполнить модалку из мантин в папке страницы
There was a problem hiding this comment.
Выше описала на странице по этому компоненту, что можно взять модалку мантин(это проще). Чем писать свою логику. Плюс в таблицу напротив каждого юзера можно вставить кнопку компонент IconButton и при нажатии передавать типизированный обьект юзер(не разбирать отдельно на много пропов в компонент, так в целом это антипаттерн). Обращаться уже потом user.name, user.contactInfo и т.д
| @@ -0,0 +1,31 @@ | |||
| import { Container, Text, Button, Group, Box } from '@mantine/core'; | |||
|
|
|||
| interface BlacklistWidgetProps { | |||
There was a problem hiding this comment.
Тут скорее всего нужно сделать интерфейс UserInformation. И в пропах передавть обьекст юзера
|
|
||
| interface RemoveFromBlacklistProps { | ||
| name: string; | ||
| onConfirm: () => void; |
There was a problem hiding this comment.
в мантин есть готовое модальное окно + хуки
| export {Header} from './header/Header.tsx'; | ||
| export {Map} from './map/Map.tsx'; No newline at end of file | ||
| export {Map} from './map/Map.tsx'; | ||
| export {BlacklistWidget} from './blacklist/AddBlackList.tsx'; |
There was a problem hiding this comment.
лучше перенести в ui страницы блеклист(они только к ней относятся)
There was a problem hiding this comment.
это не нужно отслеживать гитом(в гитигнор добавить) и надо удалить
There was a problem hiding this comment.
это не нужно отслеживать гитом(в гитигнор добавить) и надо удалить
There was a problem hiding this comment.
это не нужно отслеживать гитом(в гитигнор добавить) и надо удалить
There was a problem hiding this comment.
это надо откатить, конфиги все настроены уже
2 participants
No description provided.