Skip to content

Feat/buyer order packaging fee - #100

Merged
876908 merged 25 commits into
mainfrom
feat/buyer-order-packaging-fee
Aug 3, 2026
Merged

Feat/buyer order packaging fee#100
876908 merged 25 commits into
mainfrom
feat/buyer-order-packaging-fee

Conversation

@876908

@876908 876908 commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Summary

Type of Change

  • feat: new feature
  • fix: bug fix
  • refactor: code refactor (no functional change)
  • proto: proto definition change
  • ci: CI/CD configuration
  • chore: dependency update, config change

Affected Services

  • internal/pkg
  • userservice
  • catalogservice
  • paymentservice
  • spotservice
  • errandservice

Proto Changes (if any)

  • Confirmed backward compatible (buf breaking check passed)
  • Contains breaking change — reason and migration plan described above

Checklist

  • make lint passes locally
  • New error paths are handled explicitly
  • .env.example updated if new config vars are added

Hunger61 and others added 23 commits August 1, 2026 15:48
DistributingItem 已新增 purchased_quantity 字段(proto + SQL + Row),
但 service 组装处漏了透传,前端收到 nil 导致实购量显示 0、
未采购判断失效。补齐组装透传。
Co-authored-by: yulian-art <1119253058@qq.com>
fix errand payment totals and cancel rollback
- proto: BuyerErrandOrderProductItem 新增 packaging_fee_share_cents (field 10)
- repo: 新增 CountDistinctPurchasers 统计任务下唯一付款人数
- service: BuyerOrderProductItem 加字段,calcPackagingFeeShare 按人头分摊
- handler: 映射到 proto
- proto: BuyerErrandOrderProductItem 新增 packaging_fee_share_cents (field 10)
- repo: 新增 CountDistinctPurchasers 统计任务下唯一付款人数
- service: BuyerOrderProductItem 加字段,calcPackagingFeeShare 按人头分摊
- handler: 映射到 proto
@876908
876908 requested a review from a team August 3, 2026 12:53
@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown

The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedAug 3, 2026, 1:38 PM

Hunger61
Hunger61 previously approved these changes Aug 3, 2026
@876908
876908 merged commit 8d30ae5 into main Aug 3, 2026
19 checks passed
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.

5 participants