страница организаций

effektivnayarabota1 commented 2 months ago

С чем связан запрос на фичу?

получение данных для одной из 2х страниц; продолжаем работу над бэком

Расскажите как вы это себе видите

[x] получение списка организаций, в которых пользователь имеет права администратора
- [x] можно получать список всех организаций авторизированного пользователя, но если пользователь не емеет прав на добавление новых сотрудников, то делать кнопку disabled
[x] добавление сотрудника в организацию (модальное окно)
- [x] изменить компонент на github-search-users-input. функционал текущего компонена не состыковывается с необходимым
- [x] откуда приходит список сотрудников (общий список всех сотрудников)?
- [x] что такое команды организации (см. скриншот)?
[x] получение списка сорудников в оргиназации (модальное окно)
[x] удаление сотрудника из организации (кнопка в модальном окне^)
- не могу найти graphql endpoint
- если не найду в итоге, можно использовать rest, он гарантированно есть
[x] удаление только, если у вьювера есть права администратора
[x] выводить ошибки при запросах на клиент
- [x] добавить компоненту error-message - reducer.
[x] попробовать вынести вывод ошибок на самый высокий уровень. (ui свойство в объекте ошики)
[x] оповещать юсера об успешном выполнении запроса
- [x] один вариант - изменение стиля кнопки в модальном окне (приглашение юсера)
- [x] второй вариант - тост нотификейшн синего цвета. дизайн нарисовали. (при удалении юсера)
[x] рефакторинт

Definition of done (критерий готовности)

все готово

Приложите пример реализаций

No response

Приложите материалы задачи

No response

ssagg commented 2 months ago

@effektivnayarabota1 привет. Если тебе понадобятся запросы пользователей.

вместо логина - название оганизации query FindUsersInOrg { organization(login: "ssagg-test") { membersWithRole(first: 100, after: null) { nodes { name email avatarUrl id } } } }

еще через команды можно. но, тольк если эти команды есть и пользователь в команде. Правда потом данные надо отфильтровать от дублей пользователей, если один пользователь в несколькиз командах.

query FindUsersInTeams { organization(login: "ssagg-test") { teams(first: 100, after: null) { nodes { members { nodes { avatarUrl name id organizations(first: 100) { nodes { id name avatarUrl } } } } name } } } }

effektivnayarabota1 commented 2 months ago

@ssagg ты же запрашиваешь данные по ВСЕМ СОТРУДНИКАМ, ВСЕХ ОРГАНИЗЦИЙ, в которых у ВЬЮВЕРА есть права на это?

@Nelfimov поиск, тот что на скриншоте, я делаю по всем гитхабовским профилям, верно (глобальный поиск)? Соответственно делаю на клиенте, в UI ограничиваю вывод на несколько штук (если делать поиск исключительно по УЧАСТНИКАМ НАШИХ ОРГАНИЗАЦИ, то нового пользователя нужно будет добавлять в организацию вручную через ГИТХАБ)

Nelfimov commented 2 months ago

поиск, тот что на скриншоте, я делаю по всем гитхабовским профилям, верно (глобальный поиск)?

Да

Только учтите что у гитхаба есть ограничение по graphql

effektivnayarabota1 commented 2 months ago

@Nelfimov подскажи пожалуйста как лучше работу с куки на клиенте организовать, придумал 3 варианта:

разбирать строку, которую получаем c document.cookie при помощи js'а
подключить библиотеку для работы с куки на клиенте, например вот эту
т.к. у нас куки назначаются только на сервере, можно в пропсах или через контекст передвавать с самой первой серверной функцией - root layout server - там же, где данные вьювера с сервера возвращаем

Nelfimov commented 2 months ago

@effektivnayarabota1 если выбирать между библиотекой и нативом, я предпочитаю натив пока это не усложнит мне жизнь. То есть зависит от объёма дополнительного кода.

Третий вариант у тебя не понял о чем.

@aliceinapple присоединяйся к беседе

effektivnayarabota1 commented 2 months ago

удаление сотрудника из организации

меняю graphQl запрос на Rest, не нашел ендпоинт

ошибка при запросе. разбираюсь

{
"message": "Not Found",
"documentation_url": "https://docs.github.com/rest/orgs/members#remove-an-organization-member",
"status": "404"
}

aliceinapple commented 1 month ago

@effektivnayarabota1

Тут вроде было сообщение Разобрался?

effektivnayarabota1 commented 1 month ago

@effektivnayarabota1

Тут вроде было сообщение Разобрался?

вообще да, было.

вывожу ошибки на клиент, делал через контекст и редьюсер. в одном из компонентов useContext возвращает null, вместо функции редьюсера, спрашивал об этом.

нашел сторонний пакет с таким функционалом и лаконичным интерфейсом, буду менять эту громадину на него

Nelfimov commented 1 month ago

@effektivnayarabota1 какой статус?

effektivnayarabota1 commented 1 month ago

@effektivnayarabota1 какой статус?

вчера закончил, ревью запросил

atls / team-management