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

feat(mis): 白名单账户新增有效期字段,到期自动移除 #1243

Merged
merged 13 commits into from
May 16, 2024

Conversation

usaveh
Copy link
Contributor

@usaveh usaveh commented May 9, 2024

getWhitelistedAccounts 新增返回字段 expirationDate,whitelistAccount 新增字段 expirationDate,在 getWhitelistedAccounts 新增每次查询会检测 中是否有账户过期,有的话会自动删除
1.添加白名单时样式:
image
2.白名单用户列表新增过期时间一列
image

Copy link

changeset-bot bot commented May 9, 2024

🦋 Changeset detected

Latest commit: d83a71d

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

This PR includes changesets to release 16 packages
Name Type
@scow/mis-web Patch
@scow/mis-server Patch
@scow/demo-vagrant Patch
@scow/auth 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/audit-server Patch
@scow/portal-server Patch
@scow/portal-web Patch
@scow/ai Patch
@scow/gateway Patch
@scow/cli Patch

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 9, 2024
@pkuhpc-review-bot pkuhpc-review-bot bot requested a review from ddadaal May 9, 2024 08:45
apps/mis-server/src/entities/AccountWhitelist.ts Outdated Show resolved Hide resolved
apps/mis-server/src/entities/AccountWhitelist.ts Outdated Show resolved Hide resolved
protos/server/account.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 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 12:25
@usaveh usaveh changed the title test: 白名单账户新增有效期字段,到期自动移除 feat: 白名单账户新增有效期字段,到期自动移除 May 11, 2024
@usaveh usaveh changed the title feat: 白名单账户新增有效期字段,到期自动移除 feat(mis): 白名单账户新增有效期字段,到期自动移除 May 13, 2024
.changeset/flat-students-walk.md Outdated Show resolved Hide resolved
apps/mis-server/src/services/account.ts Outdated Show resolved Hide resolved
apps/mis-web/src/i18n/zh_cn.ts Outdated Show resolved Hide resolved
protos/server/account.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 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 08:41
.changeset/chatty-sloths-wait.md Show resolved Hide resolved
@pkuhpc-review-bot pkuhpc-review-bot bot removed the Code-ReviewRequested Code Review Requested label May 15, 2024
@pkuhpc-review-bot pkuhpc-review-bot bot added Code-Approved Code Review approved ReadyForMerge Ready for merge labels May 15, 2024
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.

点错了,看上一个

@pkuhpc-review-bot pkuhpc-review-bot bot added Code-ChangeRequested and removed Code-Approved Code Review approved ReadyForMerge Ready for merge labels May 15, 2024
@usaveh usaveh added Code-ReviewRequested Code Review Requested and removed Code-ChangeRequested labels May 16, 2024
@pkuhpc-review-bot pkuhpc-review-bot bot requested a review from ddadaal May 16, 2024 01:41
@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 16, 2024
@ddadaal ddadaal merged commit daf3885 into master May 16, 2024
9 checks passed
@ddadaal ddadaal deleted the feat-add-whitelist-time branch May 16, 2024 15:02
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.

2 participants