MerginMaps / MerginMaps/python-api-client
Check project ID when doing sync
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 23
- Forks
- 12
- Merge medio
- 2 d 13 h
- PR fusionados (30 d)
- 7
Descripción
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.
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza siguiendo el flujo de pull/push del proyecto y cómo los metadatos del proyecto almacenan actualmente el nombre del proyecto. Revisa las llamadas a la API de project-info y el issue #190; después, añade comprobaciones para la búsqueda basada en ID, el manejo de renombrados o transferencias, el borrado ante un 404 y los errores de permisos 403, manteniendo el comportamiento para los metadatos sin ID. La finalización debe incluir las pruebas automáticas descritas en el issue.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- api
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 42/100