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

Importing to directory-based journals doesn't do anything, and has incorrect message #999

Closed
wren opened this issue Jul 10, 2020 · 2 comments · Fixed by #1314
Closed

Importing to directory-based journals doesn't do anything, and has incorrect message #999

wren opened this issue Jul 10, 2020 · 2 comments · Fixed by #1314
Assignees
Labels
bug Something isn't working major Has potential for data loss 📌 This can't go stale

Comments

@wren
Copy link
Member

wren commented Jul 10, 2020

Bug Report

Environment

  • Jrnl version: v2.4.3
  • Install method: pipx
  • OS macos 10.15.3

Current Behavior

$ j -2 --export text | j dirtest --import
[2 imported to default journal]

The above should be importing to the dirtest journal, but it says it's importing to default (it's actually not importing anywhere).

Expected Behavior

Users should be able to import journal entries to directory-based journals the same way they can to single-file journals.

Repro Steps

  1. Make a directory-based journal (create an empty directory, and add to config as dirtest or whatever)
  2. Add an entry to it (e.g. jrnl dirtext some new entry) to make sure it works
  3. Try to import any number of entries to it from another journal (jrnl --export text | jrnl dirtest --import, or jrnl --import -i journal_file.txt)
  4. Observe the message saying entries were imported to default journal
  5. Observe entries were not actually imported to either journal (jrnl -1 and jrnl dirtest -1 or however you want to try to find those entries)
@wren wren added bug Something isn't working 🆕 New! and removed 🆕 New! labels Jul 10, 2020
@wren wren changed the title Importing to directory-based journals isn't working Importing to directory-based journals doesn't import anything, and has incorrect message Jul 12, 2020
@wren wren changed the title Importing to directory-based journals doesn't import anything, and has incorrect message Importing to directory-based journals doesn't do anything, and has incorrect message Jul 12, 2020
@stale
Copy link

stale bot commented Sep 11, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale Inactive issue: will be closed soon if no activity label Sep 11, 2020
@wren wren added the 📌 This can't go stale label Sep 11, 2020
@stale stale bot removed the stale Inactive issue: will be closed soon if no activity label Sep 11, 2020
@gimzmoe
Copy link

gimzmoe commented Oct 9, 2020

jrnl version v2.4.5
I was looking into trying the directory-based jrnls, but this appears to be the case for me too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working major Has potential for data loss 📌 This can't go stale
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants