-
Notifications
You must be signed in to change notification settings - Fork 2
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:新增失物招领接口 #22
Merged
Merged
feat:新增失物招领接口 #22
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
SugarMGP
reviewed
Nov 29, 2024
Penryn
force-pushed
the
dev
branch
2 times, most recently
from
December 3, 2024 02:11
87f3fb2
to
0df5413
Compare
cbluebird
reviewed
Dec 6, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- 这个pr的message是错误的,应该是rerefactor
- 这个commit在下一次一起发就行了,不是很重要🤔
Penryn
reviewed
Dec 10, 2024
- 将原有的手动拼接字符串逻辑替换为 strings.Join 函数
- 移除了 website 部分和 lostAndFound 部分之间的空行- 在 lostAndFound 部分内部添加了一个空行,以区分不同的路由定义块 - 删除了 lostAndFound 部分末尾的额外空行
- 移除 create.go 中 Imgs 字段的 required 校验 - 在 update.go 中为多个字段添加 required 校验
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.