MerginMaps / MerginMaps/python-api-client

Add some ways to work with user_email in remove_project_collaborator() and update_project_collaborator()

Aberta
#242 1 comentário 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

enhancement
Linguagem predominante
Python
Estrelas
23
Forks
12
Merge médio
2d 13h
PRs com merge (30d)
7

Descrição

When using remove_project_collaborator() or update_project_collaborator() one needs user_id which is not exactly easy to get. It can be extracted from list_project_collaborators() which provides the info, but it is not really handy or intuitive.

Either remove_project_collaborator() and update_project_collaborator() should take also user_email besides user_id to allow editing collaborators for project. Or there could be new Endpoint and client function to get user_id from user_email.

Guia de contribuição

Abrir o guia de contribuição

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Direção de pesquisa

Comece lendo remove_project_collaborator(), update_project_collaborator() e list_project_collaborators() para entender como eles lidam atualmente com user_id. Esclareça se o resultado pretendido é oferecer suporte a email nas funções existentes ou criar um endpoint de consulta de user_id e, em seguida, defina o comportamento da API e os testes necessários para demonstrar que as atualizações de colaboradores funcionam com a abordagem escolhida.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
python
Domínio
api
Tipo de issue
Funcionalidade
Dificuldade
4/5
Tempo estimado
3-5 dias
Status de atividade
Estagnada
Clareza
Precisa de esclarecimento
Facilidade para iniciantes
35/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.