-
Notifications
You must be signed in to change notification settings - Fork 11
Beta-testers needed #205
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
Comments
Hi Eryk, I am always happy to test on Windows. |
https://github.com/erykjj/jwlmanager/releases/tag/v7.0.0-beta1 I have implemented direct merge of two archives: just drag-and-drop a second one into the app while you have one opened and you will have the option to open it or merge it into the first (over-writing any at the same location and with the same title). As a result, I have adjusted some of the export and import functions (and fixed some minor issues). I need systematic testing of exporting and importing all the different categories (Notes, Highlights, Annotations, Bookmarks, Favorites). Please focus on one category at a time, and just one or two items, to ensure that the resulting archive is as expected. Thank you! |
|
I can test on iOS, iPadOS, and macOS (Apple Silicon) |
Hi @skybird-dev Thank you! Please grab the latest v7 beta from here: https://github.com/erykjj/jwlmanager/releases I can't build for iOS, but there is a macOS build. Thanks again! |
Good morning, I have done some small file merging and it is working. Working with larger files seems to hang at 40%, or perhaps I am not patient enough to wait. I have learned that when I kill the task, I have to delete the lock file for the program to start again. However, the Export to Excel is failing in 7.0 but works in 6.1.2. JWLManager v7.0.0 Traceback (most recent call last): |
Hi Glen, |
Hi @glenmwright , Both issues should be fixed in beta4. Please confirm. But you raised an interesting question: should the merge proceed despite the error? Personally, I think that it should terminate. You don't want to merge a damaged archive and you know you won't have a complete merge. On the other hand, it may be a way to "rescue" the good stuff from a corrupted archive ?? Open to opinions, please. |
While there are some clues in the Abort box, I am not sure I would be able to find the error to make the merge work. I can see it is a highlight in the Isaiah Book 1. Fortunately, you have already give us superb tools to export, delete, and import to "rescue" the good stuff. |
That's a very interesting export... You'll notice the entries starting with your highlighted one are missing A) Bible book and chapter number (first two "None" fields) AND B) DocumentId (third "None" field). The highlight has to be placed in either a Bible verse or a publication paragraph. In this case, we've got the paragraph number (second number from the left), but we haven't got the DocumentId that would identify the article/chapter in the ip-1 book. So, the question arises, how did you get those entries there? |
Hi @skybird-dev ! |
Thank you, @AnatolyOn , for your report! This was confirmed and fixed. I had been using drag-and-drop to open files and neglected to check how opening via the menu was affecte by the recent changes. Good job! |
@skybird-dev @AnatolyOn @glenmwright Any other unusual behavior or "anomalies" observed in your testing? |
@AnatolyOn |
Thank you. It's as I suspected: these are empty notes that are being removed (21 of them). That is by design: notes that have a blank title and an empty note body are removed to "clean up" the database. The solution: if you want them there as some kind of "bookmark", add something to either the title and/or body. |
v7.0.0-beta5 is out with some important fixes. Thank you all! |
Eryk, Beta 5 and your keen eye solved all my issues. I dragged a playlist in and it imported successfully. I will continue testing. |
I discovered something a couple times and I thought it was just me. This seems to be an issue with v6.1.2 as well.
|
Thank you again, @glenmwright ! |
I just pushed out v7.0.0-beta6, hopefully the last one before the official release. Please report any anomalities you may spot, especially with exporting, importing and merging. Thank you! |
Wow, @glenmwright , that's pretty bad :-| I import notes into my archive and reindex on a weekly basis, and have never lost tags, but I have confirmed that it does happen with your steps... It may have to do with it being a new archive?? I will investigate. Hope you are testing with non-official backups. |
I have tried it with a few of my backups and the tags are deleted in each one. This hasn't always been the case because I remember it working in the past. I am always careful and keep multiple backups of my backups and you have made it easier for us to restore in case of an emergency. |
I found the bug. It was introduced at some point after the Playlist feature was added... as part of the Playlist-related tables being reindexed. Glen, I really appreciate your input/colaboration! |
v7.0.0-beta7 is out with a fix |
@glenmwright |
Excellent my tags stayed, re-indexing with a new archive and my backup files work as they should. |
Closing this as v7.0.0 is now released. Please submit regular issues if you find further anomalies. |
I am available to help and learn. Portuguese language (Brazil) |
Hi @Reulym , thank you for your offer. Please feel free to report any issues you may find while using the app. Portuguese is an option in the View > Language menu. You can double-check the translations as you use it :-) |
Please let me know if you are able to collaborate by doing some beta-testing. This will include trying out new functionality and doing all kinds of things to try to break the app, and reporting the results.
I will need to know what OS you can test on.
The text was updated successfully, but these errors were encountered: