search

ya-pomogau / frontend

5 stars 7 forks source link

Проверка работы хука "usePermission" #195

Closed INextYP closed 3 weeks ago

INextYP commented 1 month ago
  1. В хук нужно обязательно добавить проверку на isRoot. Если пользователь имеет это свойство со значением true, то он может ВСЁ.
  2. Длина массива requirments должна быть равна длине hasPermission. Если да, то isAllowed = true иначе isAllowed = false
INextYP commented 1 month ago

@AlexeyKrutyakov Привет! Ветка для работы fix/use-permission команда для перехода в ветку git checkout fix/use-permission

AlexeyKrutyakov commented 1 month ago

@INextYP сделал проверку на isRoot внутри проверки на наличие роли, потому что в типе User поле role обязательное.

  1. я бы еще везде поправил requirments на requirements (нужно?) Затронет только файл хука usePermission
  2. в коммитах все пишут по-разному, кто-то на русском, кто-то на английском, на каком могу писать? (привык на английском)
INextYP commented 1 month ago

я бы еще везде поправил requirments на requirements

"Везде" - это где? Если править, то сколько модулей это затронет?

INextYP commented 1 month ago

@INextYP сделал проверку на isRoot внутри проверки на наличие роли, потому что в типе User поле role обязательное.

  1. я бы еще везде поправил requirments на requirements (нужно?) Затронет только файл хука usePermission
  2. в коммитах все пишут по-разному, кто-то на русском, кто-то на английском, на каком могу писать? (привык на английском)
  1. Можно
  2. Лучше на английском. Во всех компаниях используется именно такое соглашение
AlexeyKrutyakov commented 1 month ago

@INextYP не совсем понял правильно ли я в карточках указал пулл-реквест, но вроде так. если что - поправь меня

INextYP commented 1 month ago

@INextYP не совсем понял правильно ли я в карточках указал пулл-реквест, но вроде так. если что - поправь меня

Да, всё так) Я завтра гляну. Старайся в PR в описании писать, что было сделано, особенно, если PR громоздский)

AlexeyKrutyakov commented 1 month ago

@INextYP не совсем понял правильно ли я в карточках указал пулл-реквест, но вроде так. если что - поправь меня

Да, всё так) Я завтра гляну. Старайся в PR в описании писать, что было сделано, особенно, если PR громоздский)

подскажи пожалуйста что за PR такой, я пока не в теме

INextYP commented 1 month ago

@INextYP не совсем понял правильно ли я в карточках указал пулл-реквест, но вроде так. если что - поправь меня

Да, всё так) Я завтра гляну. Старайся в PR в описании писать, что было сделано, особенно, если PR громоздский)

подскажи пожалуйста что за PR такой, я пока не в теме

PR - pull request :)

AlexeyKrutyakov commented 1 month ago

понял, поправил описание к PR

INextYP commented 1 month ago

@AlexeyKrutyakov Привет, оставил комментарии в коде. Как будет готово - маякуй)

AlexeyKrutyakov commented 4 weeks ago

@INextYP привет, поправил