Skip to content
This repository has been archived by the owner on Jan 21, 2024. It is now read-only.

refactor: post category management #435

Merged
merged 4 commits into from
Feb 19, 2022

Conversation

ruibaby
Copy link
Member

@ruibaby ruibaby commented Feb 14, 2022

before after
image image

Signed-off-by: Ryan Wang i@ryanc.cc

Signed-off-by: Ryan Wang <i@ryanc.cc>
@ruibaby ruibaby self-assigned this Feb 14, 2022
@ruibaby ruibaby marked this pull request as draft February 14, 2022 07:51
@ruibaby
Copy link
Member Author

ruibaby commented Feb 15, 2022

Related PR halo-dev/halo#1650

Signed-off-by: Ryan Wang <i@ryanc.cc>
@ruibaby ruibaby marked this pull request as ready for review February 16, 2022 04:03
@guqing
Copy link
Member

guqing commented Feb 17, 2022

hi @ruibaby ,我测试后发现改版后对父分类设置密码后列表没有展示子分类,这应该是个bug,如下图:

改版后:
telegram-cloud-photo-size-5-6082323094942363704-y

改版前:
image

Signed-off-by: Ryan Wang <i@ryanc.cc>
Signed-off-by: Ryan Wang <i@ryanc.cc>
@ruibaby
Copy link
Member Author

ruibaby commented Feb 19, 2022

@guqing 已修复。

@ruibaby
Copy link
Member Author

ruibaby commented Feb 19, 2022

/cc @guqing
/cc @JohnNiang

Copy link
Member

@guqing guqing left a comment

Choose a reason for hiding this comment

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

LGTM

@ruibaby ruibaby merged commit 36aa658 into halo-dev:master Feb 19, 2022
@ruibaby ruibaby deleted the refactor/category-management branch February 19, 2022 13:11
@ruibaby ruibaby mentioned this pull request Feb 21, 2022
JohnNiang pushed a commit to JohnNiang/halo that referenced this pull request Mar 2, 2023
* refactor: post category management

Signed-off-by: Ryan Wang <i@ryanc.cc>

* refactor: post category management

Signed-off-by: Ryan Wang <i@ryanc.cc>

* refactor: tree data

Signed-off-by: Ryan Wang <i@ryanc.cc>

* refactor: tree data

Signed-off-by: Ryan Wang <i@ryanc.cc>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants