Skip to content
This repository has been archived by the owner on Dec 16, 2022. It is now read-only.

Chore: Apply useful patches from upstream #1

Merged
merged 10 commits into from
Aug 10, 2022

Conversation

stumpylog
Copy link
Member

@stumpylog stumpylog commented Jul 12, 2022

This PR applies the following PRs from the upstream project to here:

These bring in a few bug fixes and enhancements, as well as a few QoL changes. It doesn't fix any issues I know of paperless users encountering, but you never know.

All tests still pass, and the paperless image built to include this branch (at 81f9639) didn't have any problems.

@stumpylog
Copy link
Member Author

@paperless-ngx/backend

When folks have some time, I've applied a few patches from upstream to review. I believe this will help #1355 at the least

Copy link
Member

@shamoon shamoon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh I didn’t see this, nice! Not sure if you feel this needs specific testing since they were merged upstream?

@stumpylog
Copy link
Member Author

None of them have merged upstream. I just grabbed the patches for some pending PRs which looked useful.

I've tested an image with this version and it worked normally from what I could tell. Plus the existing test suite passes

@qcasey
Copy link
Member

qcasey commented Aug 10, 2022

Thanks for the work on this and #2, stumpylog. I imagine we will do beta tests for v1.8.1. Would you like to target these two PRs for that release?

Nothing in the code looks blatantly wrong to me but I can't say much beyond that.

@stumpylog stumpylog merged commit 9838d0d into paperless-main Aug 10, 2022
@stumpylog stumpylog deleted the apply-pull-patches branch August 10, 2022 22:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants