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.

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

fix: keep the favorites item when publishing #6256

Merged

Conversation

maxweilun1989
Copy link
Contributor

Hi @cnrpman , keep the config/publishing? is true as we discussed. Sorry for introducing some bugs for this project.

@github-actions github-actions bot added the fix label Aug 3, 2022
@tiensonqin
Copy link
Contributor

@maxweilun1989 One problem is that the favorites might contains some pages that are not public.

@maxweilun1989
Copy link
Contributor Author

@maxweilun1989 One problem is that the favorites might contains some pages that are not public.

OK, that is new to me. So we need hide favorites when it contains private pages?
Maybe keep the favorites but filter the private pages under it may be a better solution.

@tiensonqin
Copy link
Contributor

@maxweilun1989 Exactly! If all pages are public, you can get that by frontend.state/all-pages-public?, there's no need to filter, otherwise, you can get all the public pages by frontend.db.model/get-public-pages and remove those private pages.

@cnrpman cnrpman force-pushed the fix/keep_search_sideitem_when_publishing branch from f5b225d to bff8109 Compare August 11, 2022 15:28
@cnrpman
Copy link
Collaborator

cnrpman commented Aug 11, 2022

We can merge this PR first, as it's recovering the previous behavior?

Copy link
Collaborator

@andelf andelf left a comment

Choose a reason for hiding this comment

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

LGTM

@andelf
Copy link
Collaborator

andelf commented Aug 11, 2022

We can merge this PR first, as it's recovering the previous behavior?

Added a TODO #6334

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants