Skip to content
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

Suggestions & improves #32

Closed
Luchanso opened this issue Nov 2, 2020 · 4 comments
Closed

Suggestions & improves #32

Luchanso opened this issue Nov 2, 2020 · 4 comments
Labels
enhancement New feature or request

Comments

@Luchanso
Copy link

Luchanso commented Nov 2, 2020

Продублирую сюда:

Если браузер устройства не поддерживает Service Worker, то эта страница всегда будет отображаться. Ну и при первом открытии пользователи всегда будут видеть 404 страницу.

Эта проблема (context root'а или pathname) исправляется простыми двумя скриптами в 404.html и index.html.

Вообще это не единственная проблема с Service Worker, но у вас вроде отдельный домен, поэтому не должно больше вызывать сложностей.

  1. Как это исправить описано тут
  2. За место mobx можно сначала попробовать концепцию @redux/toolkit (но это всего лишь предложение, решать вам)
  3. Есть жирные компоненты без memo
  4. В целом проект не плохо написан, вы молодец, продолжайте в том же духе!)
@Luchanso
Copy link
Author

Luchanso commented Nov 2, 2020

  1. В таких местах есть интересный кейс - что если запрос не успел завершиться, а пользователь делает действие которое вызывает повторно запрос? Интересен этот кейс ещё тем, что второй запрос может завершиться раньше первого. Но это мелкий баг, на который маловероятно что наступят

@Luchanso
Copy link
Author

Luchanso commented Nov 2, 2020

  1. Старайтесь меньше пользоваться конструкциями const initialState = {} as State, лучше написать const initialState:State= {}, а уже в интерфейсе State сделать поля опциональными через знак вопроса: State = { blabla?: number; }

@Luchanso
Copy link
Author

Luchanso commented Nov 2, 2020

  1. Команду lint нужно запускать не только на CI, но и на прекомит хуке, чтобы она отформатировала весь код. Для прекомит хуков можно использовать Husky

@jarvis394 jarvis394 added the enhancement New feature or request label Nov 3, 2020
@jarvis394
Copy link
Owner

jarvis394 commented Nov 9, 2020

  1. Как это исправить описано тут

Fixed in fd4da60 (great thanks for your source!)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants