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

gas tank #23

Merged
merged 25 commits into from
Jun 4, 2024
Merged

gas tank #23

merged 25 commits into from
Jun 4, 2024

Conversation

cherry-yl-sh
Copy link
Member

No description provided.

@cherry-yl-sh cherry-yl-sh added the enhancement New feature or request label May 13, 2024
@cherry-yl-sh cherry-yl-sh self-assigned this May 13, 2024
Copy link

render bot commented May 13, 2024

@cherry-yl-sh cherry-yl-sh requested review from fanhousanbu and removed request for jhfnetboy May 29, 2024 06:24
Copy link
Contributor

@fanhousanbu fanhousanbu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

总结:

  1. 封装事务的用法
  2. 仅仅为返回一个类型,建议直接用匿名结构体
  3. 建议按http code的定义:2XX表示成功;3XX表示重定向;4XX表示传入方有问题;5XX表示响应方有问题

@cherry-yl-sh cherry-yl-sh merged commit f0b4f58 into main Jun 4, 2024
2 checks passed
@cherry-yl-sh cherry-yl-sh deleted the gas_tank_v1 branch June 4, 2024 12:59
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
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants