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

Tags aren't saved in XML when editing DayOne entries, and can't be searched for afterward #1049

Closed
micahellison opened this issue Sep 26, 2020 · 3 comments · Fixed by #1648
Closed
Labels
bug Something isn't working day one Issues related to Day One (dayoneapp.com) 📌 This can't go stale ready for pr Okay to start work. Feel free to ask questions.
Milestone

Comments

@micahellison
Copy link
Member

Bug Report

Environment

  • Jrnl --diagnostic output:
jrnl: v2.4.5
Python: 3.7.5 (tags/v3.7.5:5c02a39a0b, Oct 15 2019, 00:11:34) [MSC v.1916 64 bit (AMD64)]
OS: Windows 10
  • Install method: develop branch

Current Behavior

When you edit an entry in a DayOne journal and add a tag, the tag isn't saved in the DayOne XML tags section, and you can't find that entry by searching for that tag, either.

Expected Behavior

Repro Steps

With a DayOne journal:

  • Run jrnl and create an entry with no tags:
I am adding an entry
  • Run jrnl -1 --edit and add a tag:
[2020-09-26 11:37 ] I am adding an entry.

Now here is a @newtag

# 64291462002711EBB2735882A894A7CA
  • Search for the tag you added:
jrnl @newtag

The console outputs a couple line breaks and that's it. It should be showing that entry.

Other Information

The tags are included as expected when creating new entries. It's just a problem with the edit functionality.

@micahellison micahellison added bug Something isn't working day one Issues related to Day One (dayoneapp.com) 🆕 New! labels Sep 26, 2020
@wren wren added 📌 This can't go stale and removed 🆕 New! labels Oct 10, 2020
@wren wren added this to the Backlog milestone Apr 3, 2021
@jonakeys
Copy link
Contributor

When trying to reproduce this with v3.0, the edited/added tag shows up as expected. Maybe it's fixed between v2.4.5 and now with other PR's?

@micahellison
Copy link
Member Author

When trying to reproduce this with v3.0, the edited/added tag shows up as expected. Maybe it's fixed between v2.4.5 and now with other PR's?

That's great news! Thanks for checking that.

I think this issue still needs a running test to be closed, just so we can be sure that we won't have a regression.

@micahellison micahellison added the ready for pr Okay to start work. Feel free to ask questions. label Sep 3, 2022
@jonakeys
Copy link
Contributor

jonakeys commented Dec 2, 2022

When reading more diligent I saw that it's specifically about DayOne entries and there the error still occurs indeed. Creating a PR for that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working day one Issues related to Day One (dayoneapp.com) 📌 This can't go stale ready for pr Okay to start work. Feel free to ask questions.
Projects
None yet
3 participants