MerginMaps / MerginMaps/python-api-client

Check project ID when doing sync

Offen
#191 0 Kommentare 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 doing project pull/push, we should check project info based on project's ID instead of project name, to make sure that we're do the right thing even if something happened to the project:

  • project got renamed (or project was transferred to other workspace and we still have access to it) - we sync as usual, using the new project name when doing upload/download API calls
  • project was deleted - if project info returns 404, we know it got deleted
  • project access permission was removed from our user - if project info returns 403, we know there's permission problem

Note: some projects created/updated with an old version of MM plugin will not have project ID stored in the metadata - it would be good if we could handle those correctly as well (just hope that the project was not renamed / transferred).

Once #190 is implemented, it should be possible to add auto tests that will exercise that check.

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

Beginne damit, den Pull/Push-Ablauf des Projekts nachzuverfolgen und zu untersuchen, wie die Projektmetadaten derzeit den Projektnamen speichern. Prüfe die project-info-API-Aufrufe und Issue #190 und füge anschließend Prüfungen für die ID-basierte Suche, die Behandlung von Umbenennungen oder Übertragungen, die Löschung bei 404 sowie Berechtigungsfehler bei 403 hinzu, wobei das bisherige Verhalten für Metadaten ohne ID erhalten bleibt. Die Fertigstellung sollte die im Issue beschriebenen automatischen Tests umfassen.

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
Größtenteils klar
Anfängerfreundlichkeit
42/100

Neue Issues direkt in Ihr Postfach

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