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

Add Clear from All Files command in the Side Bar #508

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

imyuanx
Copy link

@imyuanx imyuanx commented Jan 26, 2022

I not have bookmarks-core so I can't test and i don't know if it's normal.

@alefragnani
Copy link
Owner

Hi @yunying1 ,

First of all, thank you for your PR.

As discussed in #509, please open an issue about the feature you are proposing, so we can gather feedback from other users.

Thank you

@alefragnani
Copy link
Owner

Hi @yunying1 ,

You don't need to close the PR. Simply open the issue, and let's open the discussion. If there is no change in the core repo, we could use the PR as the full solution.

Just change it to a Draft PR

Thank you

@imyuanx
Copy link
Author

imyuanx commented Feb 8, 2022

Hi @yunying1 ,

You don't need to close the PR. Simply open the issue, and let's open the discussion. If there is no change in the core repo, we could use the PR as the full solution.

Just change it to a Draft PR

Thank you

Thank you for your tolerance for novices!

@alefragnani alefragnani linked an issue Feb 8, 2022 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE] - Add Clear from All Files command in the Side Bar
2 participants