Shopify / Shopify/shopify_python_api

Get rid of Class variable storage and move to instance storage

Abierto
#191 3 comentarios 9 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

feature request
Lenguaje dominante
Python
Estrellas
1.4k
Forks
388
Merge medio
5 h 39 min
PR fusionados (30 d)
1

Descripción

I'm building an admin/monitoring server that handles requests from both out production and qa Shopify store.
In the current setup, the user/password/site_uri are stored at the class level, so I can't create a prod session and a qa session and store them in memory.
For every request to the REST server I have to clear the current class variables and set for requested environment.

Additionally, this is not thread safe. If two REST requests arrive at the same time for different endpoints, they will clobber each other.

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Empieza rastreando dónde se almacenan user, password y site_uri y cómo se accede a ellos para las solicitudes REST; después, revisa cómo se crean y seleccionan las sesiones para distintos entornos de Shopify. Se considera terminado cuando las sesiones de producción y QA pueden coexistir en memoria y las solicitudes concurrentes ya no sobrescriben la configuración de las demás; añade cobertura para ambos casos o ejecútala.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
python
Área
api, backend
Tipo de issue
Refactorización
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Estancado
Claridad
Necesita aclaración
Aptitud para principiantes
25/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.