search

IliaPavl / copy

0 stars 0 forks source link

LocalServise.js #1

Open kurockett opened 1 year ago

kurockett commented 1 year ago

https://github.com/IliaPavl/copy/blob/f87c34d8d097b07b4916908fc747d928e38ac3e6/src/servise/httpServise/LocalServise.js#L61

if (localStorage.getItem(ACCESS_TOKEN) === null || localStorage.getItem(REFRESH_TOKEN) === null || localStorage.getItem(AUTH) === null || localStorage.getItem(USERNAME) === null )

Проще заменить на !localStorage.getItem(ACCESS_TOKEN) || !localStorage.getItem(REFRESH_TOKEN) || !localStorage.getItem(AUTH) || !localStorage.getItem(USERNAME)

kurockett commented 1 year ago
IliaPavl commented 1 year ago

Спасибо, можно поточнее про  on? Я хз что это и как.

Отправлено из Mail.ru для Android среда, 05 апреля 2023г., 18:15 +03:00 от Nikita @.*** :

  • файлы с реакт компонентами лучше писать с расширением jsx, а не js

  • иногда страдают нейминги(как названия функций, так и коммитов), но в целом ок
  • все файлы с компонентами начинаются с большой буквы(исключение если ты пишешь их в index.jsx)
  • https://github.com/IliaPavl/copy/blob/f87c34d8d097b07b4916908fc747d928e38ac3e6/src/components/Settings/Amo.js#L122 стоит написать отдельный хендлер, чтобы в on[любой ивент] была функция. — Reply to this email directly, view it on GitHub , or unsubscribe . You are receiving this because you are subscribed to this thread. Message ID: @ github . com>
kurockett commented 1 year ago

onClick={someHandler} вместо onClick={() => some}