Shopify / Shopify/shopify_python_api
Get rid of Class variable storage and move to instance storage
Dieses Issue hat noch niemand übernommen.
- 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
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Ö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