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.

Sign up for GitHub

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

Jump to bottom

feat: Add user field to get class grades API #145

Open
wants to merge 27 commits into
base: master
Choose a base branch
from
Open

feat: Add user field to get class grades API #145

wants to merge 27 commits into from

Conversation

qianxi66
Copy link
Contributor

@qianxi66 qianxi66 commented Feb 9, 2025

No description provided.

@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 9, 2025

CLA Assistant Lite bot:
Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.

I have read the CLA Document and I hereby sign the CLA

1 out of 2 committers have signed the CLA.
✅ @qianxi66
❌ @Qianxi
qianxi seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You can retrigger this bot by commenting recheck in this Pull Request

@leoleoasd leoleoasd changed the title fix: Add user filed to get class grades API fix: Add user field to get class grades API Feb 17, 2025
@leoleoasd leoleoasd changed the title fix: Add user field to get class grades API feat: Add user field to get class grades API Feb 17, 2025
leoleoasd
leoleoasd requested changes Feb 17, 2025
View reviewed changes
base/validator/translations/zh/zh.go Outdated
@@ -52,6 +52,7 @@ var FieldTranslations = map[string]string{
"Tried": "选取尝试过题目",
"Passed": "选取通过题目",
"Token": "验证码",
"Sanitize": "是否清洗数据",
Copy link
Member

Choose a reason for hiding this comment

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

改成『是否格式化换行符』？

Copy link
Contributor Author

Choose a reason for hiding this comment

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

已修改

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@leoleoasd leoleoasd leoleoasd requested changes

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@qianxi66 @leoleoasd