-
Notifications
You must be signed in to change notification settings - Fork 20
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(information): 新增校园资讯 #56
Conversation
你把 #55 关了干啥?提到的问题你也没改完啊 |
校园资讯是确定了 专门做给 for you 发通知的是吗? |
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.
我给你一个方案,你把 Information
的类型改了(这样子可能还要改 API),改成
interface Information {
// ...other properties
publisher: {
name: string;
backgroundImageUrl: string | null;
}
}
背景图片是和发布者有直接关系的,而不是和一条推文有关,你写在这个组件里面是不合理的。这个关系应该是数据库控制的,而不是前端在代码里面绑定他们的关系。
这次给你留 Comment,你们对这种逻辑不关心的话就可以直接合并了 @Tianci-King
新增了详情查询页面 |
|
🥹, |
🥹🤪 |
你看看两个 |
啊 我最近都只是提交src文档 这俩文件还被修改了吗🥹🥺 |
这个 pr 对应的好几个 commit,你之前改的。你再发个 commit 改回来应该就行了 |
🫡 |
这下 是不是真的拿捏了 🥲 |
pr 的 title 你改一下,有些时候合并用 squash,要重新写 commit message 的 |
没太明白,是改成squash?还是要改成什么🤔 |
squash 是合并的一种方式,就是把所有 commit 合成一条新的。所以要写 commit message,内容是总结 pr 干了什么。pr 干了什么应该用一句话在 title 描述,你发 pr 的时候没注意。合并之前你先把 title 改了 |
你这 pr 里面 commit 改糊了,美观起见我选择用 squash。squash 有个坏处就是所有细节没了,如果有多个开发者,那每个开发者干了什么也不知道了。所以尽量 commit 写规范,不重要的代码修改合并在重要的 commit 里面,你在自己的仓库处理好。 你这个 pr 就是后面写的代码有点小问题,应该自己在本地 rebase 改好再 force push 上来的。如果这样用 merge 是最好的(pr 中所有 commit 保留,同时会追加一条 merge commit),这次方便起见我直接在网页上 squash 了。 @Tianci-King 合并的时候注意这些 |
还有 pr 的标题和 issue 的标题没有格式一说,把内容描述一句话概括就行了,详细的描述写在 message body 里面(发pr 的时候应该有个大 textarea) |
No description provided.