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

fix(mis&portal): 表格新增对列进行排序 #1238

Merged
merged 33 commits into from May 15, 2024
Merged

Conversation

usaveh
Copy link
Contributor

@usaveh usaveh commented May 6, 2024

新增了portal、mis中大部分表格的列排序功能

为 getOperationLogs、 getPaginatedChargeRecords、 getJobs 三个 API 新增了 sortBy、sortOreder 参数,表示排序的列、排序方式(升序、降序、默认)

对应修改的表格:
1.操作日志:
image

2.扣费记录:
image

3.已结束的作业:
image

Copy link

changeset-bot bot commented May 6, 2024

🦋 Changeset detected

Latest commit: cfa7b35

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 17 packages
Name Type
@scow/audit-server Minor
@scow/mis-server Minor
@scow/portal-web Minor
@scow/demo-vagrant Minor
@scow/mis-web Minor
@scow/auth Minor
@scow/lib-web Minor
@scow/ai Patch
@scow/grpc-api Minor
@scow/protos Patch
@scow/lib-hook Patch
@scow/lib-operation-log Patch
@scow/lib-scheduler-adapter Patch
@scow/lib-server Patch
@scow/portal-server Minor
@scow/gateway Minor
@scow/cli Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@pkuhpc-review-bot pkuhpc-review-bot bot added the Code-ReviewRequested Code Review Requested label May 6, 2024
@pkuhpc-review-bot pkuhpc-review-bot bot requested a review from ddadaal May 6, 2024 02:20
apps/ai/src/app/(auth)/layout.tsx Outdated Show resolved Hide resolved
protos/audit/operation_log.proto Outdated Show resolved Hide resolved
apps/mis-web/src/pages/api/finance/charges.ts Outdated Show resolved Hide resolved
apps/mis-web/src/i18n/en.ts Outdated Show resolved Hide resolved
.changeset/great-eyes-hammer.md Show resolved Hide resolved
@pkuhpc-review-bot pkuhpc-review-bot bot added Code-ChangeRequested and removed Code-ReviewRequested Code Review Requested labels May 6, 2024
@usaveh usaveh added Code-ReviewRequested Code Review Requested and removed Code-ChangeRequested labels May 9, 2024
@pkuhpc-review-bot pkuhpc-review-bot bot requested a review from ddadaal May 9, 2024 06:30
Copy link
Member

@ddadaal ddadaal left a comment

Choose a reason for hiding this comment

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

pr的description也改一下,只包含这个PR里的内容

protos/audit/operation_log.proto Outdated Show resolved Hide resolved
@pkuhpc-review-bot pkuhpc-review-bot bot added Code-ChangeRequested and removed Code-ReviewRequested Code Review Requested labels May 9, 2024
@usaveh usaveh changed the title fix(mis&portal): 表格新增对列进行排序,修改了部分UI的样式 fix(mis&portal): 表格新增对列进行排序 May 10, 2024
protos/server/charging.proto Outdated Show resolved Hide resolved
protos/common/sort_order.proto Outdated Show resolved Hide resolved
@pkuhpc-review-bot pkuhpc-review-bot bot added Code-ChangeRequested and removed Code-ReviewRequested Code Review Requested labels May 10, 2024
@usaveh usaveh added the Code-ReviewRequested Code Review Requested label May 11, 2024
@pkuhpc-review-bot pkuhpc-review-bot bot requested a review from ddadaal May 11, 2024 02:28
@pkuhpc-review-bot pkuhpc-review-bot bot added Code-ChangeRequested and removed Code-ReviewRequested Code Review Requested labels May 13, 2024
@usaveh usaveh added Code-ReviewRequested Code Review Requested and removed Code-ChangeRequested labels May 15, 2024
@pkuhpc-review-bot pkuhpc-review-bot bot requested a review from ddadaal May 15, 2024 02:56
@pkuhpc-review-bot pkuhpc-review-bot bot added Code-Approved Code Review approved ReadyForMerge Ready for merge and removed Code-ReviewRequested Code Review Requested labels May 15, 2024
@ddadaal ddadaal merged commit f534377 into master May 15, 2024
9 checks passed
@ddadaal ddadaal deleted the feat-ui-update-0423 branch May 15, 2024 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code-Approved Code Review approved ReadyForMerge Ready for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants