Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

商品在庫更新 #38

Closed
kiy0taka opened this issue Jul 20, 2020 · 3 comments · Fixed by #50
Closed

商品在庫更新 #38

kiy0taka opened this issue Jul 20, 2020 · 3 comments · Fixed by #50
Assignees
Labels
Milestone

Comments

@kiy0taka
Copy link
Contributor

kiy0taka commented Jul 20, 2020

概要

  • 商品規格情報の在庫数を更新

パラメータ

Name type Description
code string 商品コード (必須)
stock Int 商品規格の在庫数 (在庫無制限が無効の場合は必須)
stock_unlimited Boolean 商品規格の在庫無制限 (必須)

実行例

# 在庫無制限ではない商品
mutation {
    editProductStock (
        code: "product-001"
        stock: 5
        stock_unlimited: false
    ) {
        code
        stock
        stock_unlimited
    }
}
# 在庫無制限の商品
mutation {
    editProductStock (
        code: "product-002"
        stock_unlimited: true
    ) {
        code
        stock
        stock_unlimited
    }
}

商品コードで一意にならない場合は、エラーとする。

@kiy0taka kiy0taka added this to the 1.0 milestone Jul 20, 2020
@okazy
Copy link
Contributor

okazy commented Jul 22, 2020

在庫数を更新する場合、多くの場合は stock_unlimited = false だと予想できる。
stock_unlimited のデフォルトを false にして以下でも更新可能にすべきか。
懸念点としては stock_unlimited = true だった場合に上書きしてしまうこと。

# 在庫無制限ではない商品
mutation {
    editProductStock (
        code: "product-001"
        stock: 5
    ) {
        code
        stock
        stock_unlimited
    }
}

@okazy
Copy link
Contributor

okazy commented Jul 22, 2020

または、 stock = null でunlimitedとみなすUIも考えられる。

ProductStock には stock_unlimited が存在せず、無制限の時は null となる。

# 在庫無制限の商品
mutation {
    editProductStock (
        code: "product-001"
        stock: null
    ) {
        code
        stock
        stock_unlimited
    }
}

@okazy
Copy link
Contributor

okazy commented Jul 22, 2020

無制限は明示的に指定した方が良いとの意見をいただきましたので当初の仕様で実装します。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants