Pull request for review 2 #76
Conversation
| getUserInformation() | ||
| .then((user) => { | ||
| setIsLoggedIn(true); | ||
| if (user.role === 'volunteer') { |
There was a problem hiding this comment.
Лучше в таких случаях использовать switch case, а не if
| first_name: '', | ||
| second_name: '', | ||
| last_name: '', | ||
| const [currentUser, setCurrentUser] = useState({ |
There was a problem hiding this comment.
Не думали завести централизованное хранилище Redux или Mobx?
| Promise.all([getNews(), getPlatformAbout()]) | ||
| .then(([dataNews, dataPlatformAbout]) => { | ||
| setNews(dataNews.results); | ||
| const { about_us: aboutUs, valuations } = dataPlatformAbout; |
There was a problem hiding this comment.
Существует такой паттерн программирования называется адаптер, вот чтобы не деструктурировать с переименованием. Можете его применить кога вас не удовлетворяют поля которые приходят с сервера https://doka.guide/js/design-patterns-structural/
| closeModal(); | ||
| logOut() | ||
| .then(() => { | ||
| localStorage.removeItem('token'); |
There was a problem hiding this comment.
Вынесите работу с localStorage в отдельные функции, где ключ вы будете передавать как аргумент.
|
|
||
| const router = createBrowserRouter([ | ||
| const router = createHashRouter([ |
There was a problem hiding this comment.
Почему используете hash router? Если хотите публиковать на githubPages можете завести переменную окружения и если деплой идет на github pages то подставлять hashrouter, а вообще лучше использовать brouserRouter
| }} | ||
| /> | ||
| <Footer platformEmail={platformEmail} /> | ||
| <Modal modal={modal} closeModal={closeModal} /> | ||
| {modal.isOpen && | ||
| createPortal( |
There was a problem hiding this comment.
Портал нужно создать внутри самого компонента модального окна, т.е. обернуть его в createPortal, чтобы каждый раз не подключать его
…ject-view Feature/project view
feat(projectview): changed row gap
…t-project Feature/edit project
…changed font size
Added presentation link
…style Fix/slider style
…name fix(request): change domain name
fix: changing domain, readme
build(restarting the build): restart
refactor: readme
readme, fix build
В данном Pull request находится актуальное состояние ветки Develop.