Skip to content

Conversation

@jaroel
Copy link
Member

@jaroel jaroel commented Apr 13, 2017

This allows uploading a docx and displaying a pdf version of it using pdf.js


<browser:page
for="*"
name="dv_pdf_url"
Copy link
Member

Choose a reason for hiding this comment

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

@jaroel I would like to use a less obscure URL here. Can't we just use pdf?

Copy link
Member Author

Choose a reason for hiding this comment

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

@tisto sure, I've changed it!

@tisto
Copy link
Member

tisto commented May 12, 2017

@tkimnguyen @vangheem who can we ask for a pull request review here? Anybody who could merge and do a release? If you don't have time, you could also consider granting us permissions to do a release on pypi (my username is timo).

@tisto
Copy link
Member

tisto commented May 12, 2017

FYI: We need this for an Angular 2 client that just uses c.documentviewer internally to convert other formats to PDF. Our font-end document viewer can just render PDF.

@vangheem
Copy link
Member

@tisto I've added you as a maintainer on pypi.

Feel free to merge when you feel good about the current question of browser view name.

@jaroel jaroel changed the title Persist converted pdf and publish it as pdf/dump.pdf and dv_pdf_url Persist converted pdf and publish it as pdf/dump.pdf and @@pdf May 14, 2017
@tisto
Copy link
Member

tisto commented May 14, 2017

@vangheem thank you!

@tisto tisto merged commit 4d92814 into master May 14, 2017
@tisto tisto deleted the persist-convertedpdf branch May 14, 2017 09:53
@tisto
Copy link
Member

tisto commented May 15, 2017

@jaroel we might want to merge this feature into the Plone 5 version of c.documentviewer. Though, we don't need that for our current project...

@jaroel
Copy link
Member Author

jaroel commented May 15, 2017

@tisto I've added an issue for that: #65

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.

4 participants