Skip to content

feat: multiselect transaction query#173

Merged
chenyme merged 5 commits intolinux-do:masterfrom
small-lovely-cat:feat-multiselect-transaction-query
Feb 8, 2026
Merged

feat: multiselect transaction query#173
chenyme merged 5 commits intolinux-do:masterfrom
small-lovely-cat:feat-multiselect-transaction-query

Conversation

@small-lovely-cat
Copy link
Contributor

@small-lovely-cat small-lovely-cat commented Jan 13, 2026

例行检查

  • 我已阅读并理解 贡献者公约
  • 我已阅读并同意 贡献者许可协议 (CLA),确认我的贡献将根据项目的 Apache2.0 许可证进行许可
  • 我知晓如果此 PR 并不做出实质性更改,或可被认为是为了PR被合并而提交PR的,则可能不会被合并

变更内容

使/trade界面中的类型状态筛选器支持多选 owo

前:
https://github.com/user-attachments/assets/23572ff1-124f-4e6a-bd63-e85929a99e3b

后:
https://github.com/user-attachments/assets/1daf5ad3-0d42-47fd-a097-72ad42e997e5

变更原因

原先的/trade中,查询操作的类型和状态筛选器均支持多选,但是在点击了多个选项后,实际上只有第一个选项生效,这会给用户带来困扰。

本PR补全了多选的操作,使其能够正常应对前端传递的array owo

注意

这个是 #170 的一个子分支,请先查看并合并那个PR喵 w

@chenyme chenyme requested review from chenyme and yyg-max January 16, 2026 01:44
}

// buildTypeQuery builds a subquery for multiple order types
func buildTypeQuery(baseQuery *gorm.DB, types []string, userID uint64, clientID string, clientIDHandled *bool, c *gin.Context) *gorm.DB {
Copy link
Collaborator

Choose a reason for hiding this comment

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

不能直接加in吗

Copy link
Contributor Author

Choose a reason for hiding this comment

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

(okok,周一猫猫改一下 www

Copy link
Collaborator

@yyg-max yyg-max Jan 17, 2026

Choose a reason for hiding this comment

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

好的,记得按照之前的代码改保留注释 www

Copy link
Contributor Author

Choose a reason for hiding this comment

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

好的,记得按照之前的代码改保留注释 www

改好了,佬有空再看看 owo

@github-actions
Copy link

github-actions bot commented Feb 3, 2026

此 PR 长期无活动,将在 14 天后自动关闭。如需继续讨论请回复

@github-actions github-actions bot added the Stale label Feb 3, 2026
@small-lovely-cat
Copy link
Contributor Author

喵呜 w

@github-actions github-actions bot removed the Stale label Feb 4, 2026
@chenyme chenyme merged commit 192d484 into linux-do:master Feb 8, 2026
9 checks passed
@small-lovely-cat small-lovely-cat deleted the feat-multiselect-transaction-query branch February 8, 2026 05:38
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.

3 participants