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

修复 product_spu 相关的单元测试 #513

Open
wants to merge 7 commits into
base: master-jdk17
Choose a base branch
from

Conversation

DevDengChao
Copy link
Contributor

No description provided.

@DevDengChao DevDengChao changed the base branch from master to master-jdk17 May 8, 2024 08:54
@YunaiV
Copy link
Owner

YunaiV commented May 11, 2024

感谢哈。

status 是不是不传递哈?

@YunaiV YunaiV added the enhancement New feature or request label May 11, 2024
@DevDengChao
Copy link
Contributor Author

status 是不是不传递哈?

没太看懂。

@YunaiV
Copy link
Owner

YunaiV commented May 11, 2024

商品发布的时候,不考虑 status 变化哈。

@YunaiV
Copy link
Owner

YunaiV commented May 18, 2024

已经第一轮 review,等待修改

@DevDengChao
Copy link
Contributor Author

商品发布的时候,不考虑 status 变化哈。

我之前用 vue3 的管理端创建商品时出现了由于缺少 status 导致写入 mysql 失败的问题.
不确定这个是不是由于单元测试使用的 h2 数据库的表定义与 mysql 数据库的表定义不一致导致的.

另外, 创建的商品不见得会立即上架, 也存在创建待上架的商品的场景.

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

Successfully merging this pull request may close these issues.

None yet

2 participants