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

[ZCO-137] Implement OmniSearch as optional strategy in /notes/search #59

Closed
czottmann opened this issue Apr 19, 2023 · 1 comment
Closed

[ZCO-137] Implement OmniSearch as optional strategy in /notes/search #59

czottmann opened this issue Apr 19, 2023 · 1 comment
Labels
enhancement New feature or request linear
Milestone
✅ Actions URI 1.0

Comments

@czottmann
Copy link
Owner

czottmann commented Apr 19, 2023 •
edited

Getting search results out of Obsidian involves DOM scraping and is a mess. Implement Omnisearch support in /notes/omnisearch.

ZCO-137
@czottmann czottmann changed the title Research way to search notes in Obsidian that doesn't involve DOM scraping Implement OmniSearch as optional strategy in /notes/search Apr 19, 2023
@czottmann czottmann added the enhancement New feature or request label Apr 19, 2023
@czottmann czottmann added the WIP label May 4, 2023
czottmann added a commit that referenced this issue May 4, 2023
@czottmann
[CHG] Makes community plugin check more generic
6a7deab 
[NEW] Adds `PluginResultObject`

(#59)
czottmann added a commit that referenced this issue May 4, 2023
@czottmann
[NEW] Adds getEnabledCorePlugin()
f2f7f9b 
[CHG] Refactors `doSearch()`

(#59)
czottmann added a commit that referenced this issue May 4, 2023
@czottmann
[NEW] Adds getEnabledCommunityPlugin()
037515f 
[NEW] Adds `doOmnisearch()`

(#59)
czottmann added a commit that referenced this issue May 4, 2023
@czottmann
[NEW] Adds /omnisearch routes (#59)
1df0cf4
@czottmann czottmann mentioned this issue May 4, 2023
Merged
czottmann added a commit that referenced this issue May 4, 2023
@czottmann
Merge pull request #62 from czottmann/next
b20d057 
v1.1.0

- [NEW] Adds `/omnisearch` routes (#59)
- [NEW] Adds graceful handling of unexpected exceptions in route handlers (#60)
- [FIX] Adds code for preventing a race condition in vaults w/ Templater enabled (#61)
- [FIX] Replaces deprecated `global` refs w/ `window`
- [DOC] Adds dynamic TOCs to route pages
- [DOC] Changes formatting for headlines w/ tags
@czottmann
Copy link
Owner Author

Added in v1.1.0.

@czottmann czottmann removed the WIP label May 4, 2023
@czottmann czottmann added this to the ✅ v1.0 milestone Jun 12, 2023
@czottmann czottmann added linear and removed linear labels Sep 17, 2023
@czottmann czottmann changed the title Implement OmniSearch as optional strategy in /notes/search [ZCO-137] Implement OmniSearch as optional strategy in /notes/search Sep 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request linear
Projects
None yet
Milestone
✅ Actions URI 1.0
Development

No branches or pull requests
1 participant
@czottmann