-
Notifications
You must be signed in to change notification settings - Fork 2
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
JSONDecodeError when downloading tar exclude or exporting files #50
Comments
Thanks, Shelly! Sounds like I introduced a regression with the other fix. I'll take a look and get out a second release candidate with a fix shortly. |
@ShellyYBlack Try this new release candidate when you have a chance and let me know if it works better for you: https://github.com/bulk-reviewer/bulk-reviewer/releases/tag/v0.3.1-rc2. You may need to create a new session from your existing reports, since I modified how the decoding works at a few points. Thanks! |
Many thanks, Tessa! I did have to create a new session. (When I used a previous session, I got the same JSONDecodeError when exporting files or creating the tar exclude list.) There were a couple of new errors using rc2 with the new session. Creating the tar exclude fileThe first time I did this, I got this message:
The log showed this:
I then dismissed all features except one, and when I attempted to create the tar exclude list, it was successful. Exporting filesI tried exporting private files, and got this message:
Here are the log details:
I also tried exporting cleared files, and this was the error in the log:
I hope I've described all of this clearly! |
Thanks Shelly! I'm having some health issues but should hopefully be able to look into this again next week. |
Thanks for the update @tw4l and take care! |
Hi @ShellyYBlack - apologies for the long delay on this one but I have a new release candidate that I think should resolve this issue if you'd like to test: https://github.com/bulk-reviewer/bulk-reviewer/releases/tag/v0.3.1-rc3 |
Thank you @tw4l! I tested creating the exclude file and exporting files successfully. I should note, though, that I tested using new files. (I no longer had access to the files I used when encountering the errors in the previous release.) |
That's excellent @ShellyYBlack - thank you for testing! I will cut a new 0.3.1 release ASAP that will functionally be the same as v0.3.1-rc3 except for the version number. |
Hello! When I attempt to either download a tar exclude file or export files, I get this error:
I'm using release candidate 0.3.1 and working with the same files which led to that other issue you recently fixed.
The text was updated successfully, but these errors were encountered: