-
Notifications
You must be signed in to change notification settings - Fork 3
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
Various fixes #4
base: main
Are you sure you want to change the base?
Conversation
bcotton
commented
Dec 31, 2022
- Fixed cross-platform file path handling
- Added ability to handle 301 and 302 redirects
- small fix for permaLink and UTF8 characters
Thank you so much for all these contributions, all of them look like great improvements. I'll go through them individually and merge these soon. |
Unsure, I'm testing on a mac. My path does not have a trailing slash. |
I did various tests today, and I can confirm that the pathing change works well and is backward compatible. But I did face a problem and it's 100% repeatable for me right now. This error happens each time when downloading .json
I checked the saved.json and it's always truncated at the same spot The full saved.json shows this data right after the truncation Any idea what's causing the problem? |
I'm having similar problems, but it will succeed about 25% of the time. It's something to do with the timing of the Promise, dunno. Still looking into it. |
For me it was doing that 100% of the time yesterday at one point. Since then I have unsaved the latest posts/comments and saved some new ones and I can't recreate it right now - I wish I didn't make a change so I could keep looking into it, but my current json download doesn't give me an error. |
I pushed a new commit. This works for me, every time. |
Thank you for the update - pls allow me a couple days to use/test it and I'll merge with the main branch. |
I have been using your updated version for last few days and have seen no issues whatsoever. Just created a new release. |