Shopify / Shopify/shopify_python_api

Get rid of Class variable storage and move to instance storage

未关闭
#191 3 条评论 9 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

feature request
主要语言
Python
星标
1.4k
派生
388
平均合并
5 小时 39 分钟
30 天内合并 PR
1

描述

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.

贡献指南

打开贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

调研方向

首先跟踪 user、password 和 site_uri 在哪里存储,以及 REST 请求如何访问它们,然后检查如何为不同的 Shopify 环境创建和选择 session。完成的标准是 production 和 QA session 可以在内存中共存,并且并发请求不再相互覆盖配置;为这两种情况添加或运行 coverage。

由索引模型根据 Issue 内容生成。

评估

技术栈
python
领域
api, backend
Issue 类型
重构
难度
4/5
预计耗时
3-5 天
活跃度
停滞
描述清晰度
需要澄清
新手友好度
25/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。