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

Can't list notes by tags #11

Closed
jessmartin opened this issue Sep 30, 2020 · 4 comments · Fixed by #13
Closed

Can't list notes by tags #11

jessmartin opened this issue Sep 30, 2020 · 4 comments · Fixed by #13
Assignees

Comments

@jessmartin
Copy link

Hey @a5huynh - great project!

Unfortunately, I'm not able to list notes by tag due to the tags-to-notes join table having a different name than expected. See this ticket for a detailed explanation, along with probable fix: calebporzio/bear-sync#4

Here's what I did:

% RUST_BACKTRACE=1 cub ls -t writing/drafting
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: SqliteFailure(Error { code: Unknown, extended_code: 1 }, Some("no such table: Z_6TAGS"))', src/libcub/mod.rs:132:28
stack backtrace:
   0: <std::sys_common::backtrace::_print::DisplayBacktrace as core::fmt::Display>::fmt
   1: core::fmt::write
   2: std::io::Write::write_fmt
   3: std::panicking::default_hook::{{closure}}
   4: std::panicking::default_hook
   5: std::panicking::rust_panic_with_hook
   6: rust_begin_unwind
   7: core::panicking::panic_fmt
   8: core::option::expect_none_failed
   9: libcub::list_notes
  10: cub::main
  11: std::rt::lang_start::{{closure}}
  12: std::panicking::try::do_call
  13: __rust_maybe_catch_panic
  14: std::rt::lang_start_internal
  15: main
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

You can see my join table is actually called Z_7TAGS rather than Z_6TAGS:
image

I'm running BearApp 1.7.16

@a5huynh
Copy link
Owner

a5huynh commented Sep 30, 2020

@jessmartin Thanks for submitting the issue!

I can confirm this is failing locally for me as well. And that fix you linked seems to do the trick, I'll push out a new version ASAP.

@jessmartin
Copy link
Author

jessmartin commented Sep 30, 2020 via email

@a5huynh a5huynh self-assigned this Sep 30, 2020
a5huynh added a commit that referenced this issue Oct 1, 2020
🐛 Update tag filter query to work w/ latest version of Bear (1.7.16)
@a5huynh
Copy link
Owner

a5huynh commented Oct 1, 2020

@jessmartin could you confirm everything is working w/ the latest version? Just pushed out the update. Thanks!

@jessmartin
Copy link
Author

jessmartin commented Oct 5, 2020 via email

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 a pull request may close this issue.

2 participants