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

添加了查成绩过滤器的记忆功能 #14

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

KraHsu
Copy link

@KraHsu KraHsu commented Jun 28, 2023

添加了查成绩过滤器的记忆功能,来自群友的想法

添加了查成绩过滤器的记忆功能,来自群友的想法
@vercel
Copy link

vercel bot commented Jun 28, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
bit-101 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 29, 2023 1:35am

Comment on lines +311 to +315
<n-input :input-props="{ id: 'sid' }" v-model:value="user.sid" type="number" placeholder="学号" />
<n-input :input-props="{ id: 'password' }" v-model:value="user.password" type="password" show-password-on="click"
placeholder="学校统一身份认证密码" />
<n-button id="submit" @click="WebvpnVerify(user.sid, user.password)"
:disabled="!user.sid || !user.password || webvpn.loading" block :loading="webvpn.loading">

This comment was marked as resolved.

Copy link
Author

Choose a reason for hiding this comment

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

是的,我使用VScode会自动格式化我的代码:D

@flwfdd
Copy link
Contributor

flwfdd commented Jun 28, 2023

非常感谢request!!但我突然想到一个问题:当先查了一个人的成绩,又查了另一个人的,但是前一个人的filter中包含了一些后一个人的filter中没有包含的选项,不知道会不会造成问题。。。等我再想想或验证一下。

@KraHsu
Copy link
Author

KraHsu commented Jun 29, 2023

非常感谢request!!但我突然想到一个问题:当先查了一个人的成绩,又查了另一个人的,但是前一个人的filter中包含了一些后一个人的filter中没有包含的选项,不知道会不会造成问题。。。等我再想想或验证一下。

是的,我测试了一下,这确实会导致问题,确实是我有欠考虑。我目前能想到的解决方案是将学号也一起存储,这样可以加以区分,但实在不是很优雅。

通过区分学号避免不同用户的过滤器相互污染,但是也许会导致存储空间变得很花
(或者如果一堆人都在同一个浏览器上查询的话可能达到存储空间限制?)
其实可以用更复杂的方式来控制,但总觉得不太优雅:D
希望大佬可以写出更好的解决方案
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