📋 Pull Request Information

Original PR: https://github.com/ciur/papermerge/pull/226
Author: @francescocarzaniga
Created: 11/26/2020
Status: ✅ Merged
Merged: 11/26/2020
Merged by: @ciur

Base: master ← Head: pipeline_fixes

📝 Commits (2)

0521161 Various fixes
0f5366e PEP8

📊 Changes

5 files changed (+76 additions, -49 deletions)

View changed files

📝 config/settings/base.py (+0 -1)
📝 papermerge/core/import_pipeline.py (+32 -15)
📝 papermerge/core/importers/imap.py (+15 -9)
📝 papermerge/core/importers/local.py (+15 -15)
📝 papermerge/core/views/documents.py (+14 -9)

📄 Description

Description

I am writing my first app that makes use of pipelines so I noticed a few integration problems. This PR fixes those issue without impacting current functionality.

Type of change

Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
This change requires a documentation update

Checklist:

I have read the Contributing file available here
I have formatted this PR according to PEP8 rules
I have commented my code, particularly in hard-to-understand areas
I have made corresponding changes to the documentation
My changes generate no new warnings
I have added tests that prove my fix is effective or that my feature works
New and existing unit tests pass locally with my changes

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/ciur/papermerge/pull/226 **Author:** [@francescocarzaniga](https://github.com/francescocarzaniga) **Created:** 11/26/2020 **Status:** ✅ Merged **Merged:** 11/26/2020 **Merged by:** [@ciur](https://github.com/ciur) **Base:** `master` ← **Head:** `pipeline_fixes` --- ### 📝 Commits (2) - [`0521161`](https://github.com/ciur/papermerge/commit/0521161098c9926c24e933ad1c70182b9cf3c9c8) Various fixes - [`0f5366e`](https://github.com/ciur/papermerge/commit/0f5366e3139955542ebe6e7fc4f693ffe5617cf5) PEP8 ### 📊 Changes **5 files changed** (+76 additions, -49 deletions) <details> <summary>View changed files</summary> 📝 `config/settings/base.py` (+0 -1) 📝 `papermerge/core/import_pipeline.py` (+32 -15) 📝 `papermerge/core/importers/imap.py` (+15 -9) 📝 `papermerge/core/importers/local.py` (+15 -15) 📝 `papermerge/core/views/documents.py` (+14 -9) </details> ### 📄 Description # Description I am writing my first app that makes use of pipelines so I noticed a few integration problems. This PR fixes those issue without impacting current functionality. ## Type of change Please delete options that are not relevant. - [x] Bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) - [ ] This change requires a documentation update # Checklist: - [x] I have read the [Contributing file available here](https://github.com/ciur/papermerge/blob/master/CONTRIBUTING.md) - [x] I have formatted this PR according to [PEP8 rules](https://www.python.org/dev/peps/pep-0008/) - [x] I have commented my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes to the documentation - [x] My changes generate no new warnings - [ ] I have added tests that prove my fix is effective or that my feature works - [x] New and existing unit tests pass locally with my changes --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

closed this issue
added the
pull-request
label

Rows
Columns

[PR #226] [MERGED] Pipeline fixes #570