MerginMaps / MerginMaps/python-api-client
Check project ID when doing sync
Dieses Issue hat noch niemand übernommen.
- 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
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Ö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