Shopify / Shopify/shopify_python_api

Get rid of Class variable storage and move to instance storage

Offen
#191 3 Kommentare 9 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

feature request
Vorherrschende Sprache
Python
Sterne
1.4k
Forks
388
Ø Merge
5 Std. 39 Min.
Gemergte PRs (30 T.)
1

Beschreibung

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.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne damit, nachzuverfolgen, wo user, password und site_uri für REST-Anfragen gespeichert und abgerufen werden, und überprüfe anschließend, wie Sessions für verschiedene Shopify-Umgebungen erstellt und ausgewählt werden. Als erledigt gilt die Aufgabe, wenn Produktions- und QA-Sessions im Speicher koexistieren können und nebenläufige Anfragen die Konfiguration der jeweils anderen nicht mehr überschreiben; füge Abdeckung für beide Fälle hinzu oder führe sie aus.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
api, backend
Issue-Typ
Refactoring
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
25/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.