Shopify / Shopify/shopify_python_api

Get rid of Class variable storage and move to instance storage

Ouverte
#191 3 commentaires 9 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

feature request
Langage dominant
Python
Étoiles
1.4k
Forks
388
Merge moyen
5 h 39 min
PR mergées (30 j)
1

Description

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.

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par retracer où user, password et site_uri sont stockés et consultés pour les requêtes REST, puis examinez comment les sessions sont créées et sélectionnées pour différents environnements Shopify. Le travail est terminé lorsque les sessions de production et de QA peuvent coexister en mémoire et que les requêtes concurrentes n’écrasent plus la configuration des autres requêtes ; ajoutez ou exécutez la couverture pour les deux cas.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
python
Domaine
api, backend
Type d'issue
Refactorisation
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
À clarifier
Accessibilité débutants
25/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.