Skip to content

[9.0][FIX] report_qweb_pdf_watermark - #154

Closed
MiquelRForgeFlow wants to merge 4 commits into
OCA:9.0from
ForgeFlow:9.0-fix-report_qweb_pdf_watermark
Closed

MiquelRForgeFlow wants to merge 4 commits into
OCA:9.0from
ForgeFlow:9.0-fix-report_qweb_pdf_watermark

Conversation

@MiquelRForgeFlow

Copy link
Copy Markdown
Contributor

Backport of #144.

from base64 import b64decode
from logging import getLogger
from pyPdf import PdfFileWriter, PdfFileReader
from pyPdf.pdf import PdfFileWriter, PdfFileReader

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why this line was changed?

@MiquelRForgeFlow MiquelRForgeFlow Jul 28, 2017

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Because otherwise my PyCharm says 'unresolved reference...' 😙

@AaronHForgeFlow

Copy link
Copy Markdown
Contributor

image
This error is showing in all the Pull Request in v9. This PR is meant to solve that but it's not working. Can anybody the label Needed help here?

@astirpe

astirpe commented Jul 28, 2017

Copy link
Copy Markdown
Member

Can you try to backport the entire commit 7d62102? I mean deleting also the obsolete rows in .travis.yml.

I'm not sure this will fix the problem though.

@MiquelRForgeFlow

MiquelRForgeFlow commented Jul 28, 2017

Copy link
Copy Markdown
Contributor Author

I fully backported the two commits of that PR but both squashed into one.

@astirpe

astirpe commented Jul 28, 2017

Copy link
Copy Markdown
Member

@mreficient it seems you missed a commit. Can you double check? Take a look at the commit mentioned in my previous post and then look at your commits.

@MiquelRForgeFlow

Copy link
Copy Markdown
Contributor Author

623f86e is a squashed version of 6daceca and 7d62102.

@hbrunn

hbrunn commented Aug 30, 2017

Copy link
Copy Markdown
Member

this finally is fixed in #141 by now

@hbrunn hbrunn closed this Aug 30, 2017
@MiquelRForgeFlow
MiquelRForgeFlow deleted the 9.0-fix-report_qweb_pdf_watermark branch September 12, 2017 12:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants