Check project ID when doing sync
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 42/100
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.
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.
- Lenguaje dominante
- Python
- Estrellas
- 23
- Forks
- 12
- Merge medio
- 2 d 13 h
- PR fusionados (30 d)
- 7
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.
Más de MerginMaps/python-api-client
-
Vacuuming Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
MerginMaps/python-api-client#320 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
MerginMaps/python-api-client#270 ·
-
Migration tool between servers Abiertoenhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
MerginMaps/python-api-client#257 · 3 comentarios ·
-
enhancement
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
MerginMaps/python-api-client#254 · 1 comentario ·
-
Impose geodiff push limits Abiertoenhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
MerginMaps/python-api-client#245 ·
Todos los issues de MerginMaps/python-api-client
Issues similares
-
货币战争手改优先级配置缺少列表元素类型校验(P3) Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
syfoud/Simulated_Scepter#172 ·
-
A cancelled tests run makes the coverage comment workflow fail and reports it as a red check on main Abiertoarea: ci bug perceived difficulty: 3
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Nitjsefnie-Harness-Commons/daedalus#921 · 1 comentario ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
EleutherAI/lm-evaluation-harness#4207 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
ClickHouse/clickhouse-connect#1057 ·