MerginMaps / MerginMaps/python-api-client

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

Offen
#242 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

enhancement
Vorherrschende Sprache
Python
Sterne
23
Forks
12
Ø Merge
2 T. 13 Std.
Gemergte PRs (30 T.)
7

Beschreibung

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.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginnen Sie damit, remove_project_collaborator(), update_project_collaborator() und list_project_collaborators() zu lesen, um zu verstehen, wie sie derzeit mit user_id umgehen. Klären Sie, ob das beabsichtigte Ergebnis E-Mail-Unterstützung in den bestehenden Funktionen oder einen user_id-Lookup-Endpunkt umfasst, und definieren Sie dann das API-Verhalten und die erforderlichen Tests, um zu zeigen, dass Aktualisierungen von Collaborators mit dem gewählten Ansatz funktionieren.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
api
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.