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

fix: fix visit bug #1381 #1414

Merged
merged 8 commits into from
Jul 12, 2021
Merged

fix: fix visit bug #1381 #1414

merged 8 commits into from
Jul 12, 2021

Conversation

yuanzhixiang
Copy link

@yuanzhixiang yuanzhixiang commented Jul 7, 2021 •
edited
Loading

Fix #1381

@guqing
Copy link
Member

guqing commented Jul 7, 2021

你好 请先修复一下代码格式确保ci的code style能够检查通过

@JohnNiang
Copy link
Member

建议在 PR 的描述中关联 issue，例如：

Fix #1381
Fix #1414

@yuanzhixiang
Copy link
Author

你好 请先修复一下代码格式确保ci的code style能够检查通过

好的， 我今天看下

@yuanzhixiang
Copy link
Author

建议在 PR 的描述中关联 issue，例如：

Fix #1381
Fix #1414

好的

@mimajiushi
Copy link

好奇测试代码要怎么解决

@guqing
Copy link
Member

guqing commented Jul 9, 2021

对于回收站的文章访问时现在返回的是403页面您没有该文章的访问权限 我认为回收站的文章应该是404，文章不存在, @JohnNiang @ruibaby 你们怎么看

@ruibaby
Copy link
Member

ruibaby commented Jul 11, 2021 •
edited
Loading

对于回收站的文章访问时现在返回的是403页面您没有该文章的访问权限 我认为回收站的文章应该是404，文章不存在, @JohnNiang @ruibaby 你们怎么看

我认为应该是404，只要不是公开的，都应该是 404。

@guqing
Copy link
Member

guqing commented Jul 11, 2021

对于回收站的文章访问时现在返回的是403页面您没有该文章的访问权限 我认为回收站的文章应该是404，文章不存在, @JohnNiang @ruibaby 你们怎么看

我认为应该是404，只要不是公开的，都应该是 404。
@zhixiangyuan 感谢你的贡献 经过讨论 我们认为对于访问不到的文章应该返回404页面目前返回的是403 需要对此做出一些修改

@yuanzhixiang
Copy link
Author

对于回收站的文章访问时现在返回的是403页面您没有该文章的访问权限 我认为回收站的文章应该是404，文章不存在, @JohnNiang @ruibaby 你们怎么看

我认为应该是404，只要不是公开的，都应该是 404。
@zhixiangyuan 感谢你的贡献 经过讨论 我们认为对于访问不到的文章应该返回404页面目前返回的是403 需要对此做出一些修改

再看下，改过了

@guqing
Copy link
Member

guqing commented Jul 11, 2021

对于回收站的文章访问时现在返回的是403页面您没有该文章的访问权限 我认为回收站的文章应该是404，文章不存在, @JohnNiang @ruibaby 你们怎么看

我认为应该是404，只要不是公开的，都应该是 404。
@zhixiangyuan 感谢你的贡献 经过讨论 我们认为对于访问不到的文章应该返回404页面目前返回的是403 需要对此做出一些修改

再看下，改过了

好的 我们会尽快review

@JohnNiang
Copy link
Member

LGTM

@guqing
Copy link
Member

guqing commented Jul 12, 2021

LGTM

@JohnNiang JohnNiang merged commit 189f7be into halo-dev:master Jul 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@guqing guqing guqing approved these changes

@ruibaby ruibaby ruibaby approved these changes

@LIlGG LIlGG Awaiting requested review from LIlGG

@JohnNiang JohnNiang Awaiting requested review from JohnNiang

Assignees

@JohnNiang JohnNiang

@ruibaby ruibaby

@LIlGG LIlGG

@guqing guqing

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

【BUG】回收站的文章可以访问
6 participants
@yuanzhixiang @guqing @JohnNiang @mimajiushi @ruibaby @LIlGG