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

Add support for exporting notes #6

Closed
OGKevin opened this issue Sep 10, 2022 · 10 comments · Fixed by #31
Closed

Add support for exporting notes #6

OGKevin opened this issue Sep 10, 2022 · 10 comments · Fixed by #31
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@OGKevin
Copy link
Owner

OGKevin commented Sep 10, 2022

Add the ability to include notes made in the export.

Relates to: bitwiseops#2

@OGKevin OGKevin self-assigned this Sep 10, 2022
@OGKevin OGKevin added the enhancement New feature or request label Sep 10, 2022
@OGKevin OGKevin added this to the 1.3.0 milestone Sep 10, 2022
@MichaelLepori
Copy link

Nice! Thank you @OGKevin

@OGKevin OGKevin moved this from 📋 Backlog to 🏗 In progress in Obsidian kobo plugin Sep 15, 2022
OGKevin added a commit that referenced this issue Sep 15, 2022
@OGKevin OGKevin linked a pull request Sep 15, 2022 that will close this issue
@OGKevin
Copy link
Owner Author

OGKevin commented Sep 15, 2022

@MichaelLepori Are you able to do a test again, before I merge the MR?

https://github.com/OGKevin/obsidian-kobo-highlights-import/releases/tag/1.3.0-beta.0

@MichaelLepori
Copy link

MichaelLepori commented Sep 15, 2022 via email

@OGKevin
Copy link
Owner Author

OGKevin commented Sep 16, 2022

@MichaelLepori
Copy link

MichaelLepori commented Sep 17, 2022 via email

@OGKevin
Copy link
Owner Author

OGKevin commented Sep 17, 2022

@MichaelLepori I think you wanted to send an attachment? I think that is not possible via email. You must do that via GitHub directly.

The code that touches ordering has no changes in this release 🤔
I don't see any order issues in my diff.
The way ordering is done is based on the chapter the note has been taken.
If you have 2 highlights within the same chapter, then the ordering can be incorrect 🤔
Is this what you are experiencing?

@MichaelLepori
Copy link

MichaelLepori commented Sep 17, 2022 via email

@OGKevin
Copy link
Owner Author

OGKevin commented Sep 17, 2022

@MichaelLepori Nope

@MichaelLepori
Copy link

@OGKevin I write hier on GitHub:
Hier the Result:
image

image

Why ist there a space more on the first line?
image

I tried Multiline on the Note of a Highlight.

2nd Question:
How does it work with the sorting?
The first example ist the last Highlight Note on the top.The second example ist the last Note at the end.

OGKevin added a commit that referenced this issue Sep 20, 2022
@OGKevin
Copy link
Owner Author

OGKevin commented Sep 20, 2022

@MichaelLepori The extra space should be fixed in https://github.com/OGKevin/obsidian-kobo-highlights-import/releases/tag/1.3.0-beta.1.

Regarding sorting, it sorts based on an internal ID that is stored in the database:

return a.content.bookTitle.localeCompare(b.content.bookTitle) ||
a.content.contentId.localeCompare(b.content.contentId);

For some reason, the ID of chapter27 is sorted above the ID of chapter1. The only way to find out why is to see what the values of the ContentID is.

If you would like, I can add debug logging in the next release that you can use to extract this information, and then we can figure out why the sorting is not working for this book.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: 🏗 In progress
2 participants