MerginMaps / MerginMaps/python-api-client
Check project ID when doing sync
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 23
- Forks
- 12
- Merge moyen
- 2 j 13 h
- PR mergées (30 j)
- 7
Description
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.
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par suivre le flux de pull/push du projet et la manière dont les métadonnées du projet stockent actuellement le nom du projet. Examinez les appels à l’API project-info et l’issue #190, puis ajoutez des vérifications pour la recherche basée sur l’ID, la gestion des renommages ou des transferts, la suppression en cas de 404 et les erreurs d’autorisation 403, tout en préservant le comportement pour les métadonnées sans ID. La finalisation doit inclure les tests automatiques décrits dans l’issue.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- python
- Domaine
- api
- Type d'issue
- Fonctionnalité
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 42/100