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

[BUG] Folder ignore sorts all folder starting with string #969

Closed
4Source opened this issue May 23, 2024 · 0 comments · Fixed by #972
Closed

[BUG] Folder ignore sorts all folder starting with string #969

4Source opened this issue May 23, 2024 · 0 comments · Fixed by #972
Labels
beta Included in beta release, https://github.com/st3v3nmw/obsidian-spaced-repetition/discussions/1011 bug Something isn't working

Comments

@4Source
Copy link
Contributor

4Source commented May 23, 2024

Describe the bug
I recently discovered that if you have brunche of folders there are starting with same text e.g. Test-1 Test-2 and you put Test- to ignore list both get ignored.

To Reproduce
Steps to reproduce the behavior:

  1. Create 2 folder ./Test-1 ./Test-2
  2. Create in each a file with test flashcard
  3. Add to folder ignore list Test-
  4. See error

Expected behavior
Only exact match of the name should be ignored

Versions (please complete the following information):

  • OS: Win10
  • Obsidian version: 1.5.12
  • Plugin version: 1.12.4
  • If on desktop, Installer version: 5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) obsidian/1.4.16 Chrome/114.0.5735.289 Electron/25.8.1 Safari/537.36

Additional context
TestFolders.zip

@4Source 4Source added the bug Something isn't working label May 23, 2024
@ronzulu ronzulu added the beta Included in beta release, https://github.com/st3v3nmw/obsidian-spaced-repetition/discussions/1011 label Jul 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
beta Included in beta release, https://github.com/st3v3nmw/obsidian-spaced-repetition/discussions/1011 bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants