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

完成admin_order_service部分 #5

Closed
wants to merge 4 commits into from
Closed

Conversation

Mo9L1
Copy link
Collaborator

@Mo9L1 Mo9L1 commented May 25, 2024

最后的端对端有一些问题,当add后,数据库明明可以查到,可是当update时,却说没有这个order!!!delete也说没这个order

Copy link

google-cla bot commented May 25, 2024

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@Lincyaw
Copy link
Owner

Lincyaw commented May 26, 2024

先去写下游的 order service 和 order other service。这个服务调用了别的服务,错误有可能是剩余的服务的,现在去 debug 那些服务比较麻烦,先去写那些没有调用其他服务的那些服务的测试,这样就会顺利一些

@Lincyaw
Copy link
Owner

Lincyaw commented May 26, 2024

另外,在提 pr 之前,用 ide 自带的格式化代码,把代码格式化一下。pycharm 是 ctrl-alt-L,vscode是Ctrl+Shift+I

@Lincyaw Lincyaw closed this May 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants