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): 平台数据统计缓存基本修复 #1336

Merged
merged 4 commits into from
Jul 5, 2024

Conversation

cuvalign
Copy link
Contributor

@cuvalign cuvalign commented Jul 3, 2024

平台管理->平台数据统计:
这些数据滞后有误且有缓存。干扰了整个页面可视图表何数据的准确性。
以账户充值为例类推

修改前

进入时

可视图表显示本周没有账户充值
本例中为2024-04-17 -> 2024-04-24
1713950395375

调整日期筛选到今天

2024-04-24 -> 2024-04-24
可见有账户充值
1713950512545
但是此时筛选回到2024-04-17 -> 2024-04-24
仍然没有账户充值(被缓存)
切换成2024-04-23 -> 2024-04-24
则有账户充值

因此,应当修改本页面,使平台数据统计数据显示正确,避免错误的缓存无法变更。

修改后

默认缓存时间为5分钟
image

Copy link

changeset-bot bot commented Jul 3, 2024

🦋 Changeset detected

Latest commit: d8f25e6

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

This PR includes changesets to release 7 packages
Name Type
@scow/mis-server Patch
@scow/auth Patch
@scow/gateway Patch
@scow/mis-web Patch
@scow/portal-server Patch
@scow/portal-web 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 Jul 3, 2024
@pkuhpc-review-bot pkuhpc-review-bot bot requested a review from ddadaal July 3, 2024 05:48
@cuvalign cuvalign removed the request for review from ddadaal July 3, 2024 05:58
@cuvalign
Copy link
Contributor Author

cuvalign commented Jul 4, 2024

请求测试接入

@cuvalign cuvalign requested a review from ddadaal July 4, 2024 09:53
apps/mis-server/src/utils/cache.ts Outdated Show resolved Hide resolved
@pkuhpc-review-bot pkuhpc-review-bot bot added Code-ChangeRequested and removed Code-ReviewRequested Code Review Requested labels Jul 4, 2024
@cuvalign cuvalign requested a review from ddadaal July 5, 2024 01:58
@pkuhpc-review-bot pkuhpc-review-bot bot added Code-ReviewRequested Code Review Requested and removed Code-ChangeRequested labels Jul 5, 2024
@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 Jul 5, 2024
@ddadaal ddadaal merged commit 67cc41c into master Jul 5, 2024
9 checks passed
@ddadaal ddadaal deleted the fix-admin-statistic-cache branch July 5, 2024 10:12
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