Conversation
|
#88 - на секунду показывается страница UserInfo |
|
Visit the preview URL for this PR (updated for commit 23998d0):
(expires Tue, 15 Dec 2020 19:04:00 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 |
|
Хмм))
UPD: Чекни этот компонент, который для HomePage и для ошибок юзается Чет с размерами тут
|
|
Пока некритично, но до показа бы исправить, что на какое-то время мелькает сама страница Т.е. в идеале должна быть сначала проверка на доступ к ресурсу (для 404), и только если ок - уже отображаем сам page (может имеет смысл какой-нить флаг accessLoading добавить на верхнем уровне, и только как она станет false - отображать чайлдов гарантированно) В рамках PR - пофиг пока, но ишью бы выписать
Да, да - поправить бы - выпиши issue себе |
azinit
left a comment
azinit
left a comment
There was a problem hiding this comment.
Все круто, но го еще добьем моменты некоторые, чтоб точно до ума довести
…o feature/error-handling � Conflicts: � src/pages/home/index.tsx � src/pages/index.tsx
| import withRouter from "./with-router"; | ||
|
|
||
| // Потом какой-нибудь `compose` метод заинсталлим откуда-нить и покрасивше будет | ||
| export const withHocs = (component: () => JSX.Element) => withRouter(withApollo(component)); |
There was a problem hiding this comment.
Некритично, но в react-app-env уже задеклейрен глобально тип для компонентов
There was a problem hiding this comment.
Так можно же props?: P выставить)
| * Обертка для подключения и работы с API | ||
| */ | ||
| const withApollo = (component: () => React.ReactNode) => () => ( | ||
| const withApollo = (component: () => JSX.Element) => () => ( |
There was a problem hiding this comment.
Некритично, но в react-app-env уже задеклейрен глобально тип для компонентов
| import withApollo from "./with-apollo"; | ||
| import withRouter from "./with-router"; | ||
|
|
||
| // Потом какой-нибудь `compose` метод заинсталлим откуда-нить и покрасивше будет |
| import { QueryParamProvider } from "use-query-params"; | ||
|
|
||
| export const setupRouter = (component: () => JSX.Element) => () => ( | ||
| const withRouter = (component: () => JSX.Element) => () => ( |
There was a problem hiding this comment.
Потом бы добавил коммент, аля
/**
* Инициализация роутера с провайдером для работы с get-параметрами
*/| import withApollo from "./with-apollo"; | ||
| import Header from "./header"; | ||
| import "./index.scss"; | ||
| import { withHocs } from "./hocs"; |
There was a problem hiding this comment.
Круто, но стили в самом низу располагают обычно)
imports/order увы не удается пока для этого настроить
| import "./index.scss"; | ||
| import { withHocs } from "./hocs"; | ||
|
|
||
| const ErrorPage = lazy(() => import("pages/error")); |
There was a problem hiding this comment.
Чет смущает, что ErrorPage тут так хардкодится... Но ладно, потом посмотрим что с этим можно сделать. Потом в след коммитах пометь как-нибудь
// !!! FIXME: manage access| q: currentTarget.value, | ||
| type: query.type, | ||
| s: query.s, | ||
| o: query.o, |
There was a problem hiding this comment.
Потом бы на такое заменил
const q = currentTarget.value;
history.push(`/search?${qs.stringify({ ...query, q })}`);2 participants
Closes #4
Closes #31