Skip to content
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

WIP: Export PDF with puppeteer #1450

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from
Open

WIP: Export PDF with puppeteer #1450

wants to merge 5 commits into from

Conversation

Yukaii
Copy link
Member

@Yukaii Yukaii commented Mar 9, 2020

It takes about 5 seconds to print a pdf now... quite slow.

TODO

  • Better download experience
  • Is this safe?
  • NPM_CONFIG_PRODUCTION true

Signed-off-by: Yukai Huang <[email protected]>
@Yukaii Yukaii temporarily deployed to codimd-feature-pdf-expo-las27r March 9, 2020 13:28 Inactive
@Yukaii Yukaii changed the title Export PDF with puppeteer WIP: Export PDF with puppeteer Mar 9, 2020
@Yukaii Yukaii added this to the Next milestone Mar 9, 2020
@Yukaii Yukaii added the enhancement Wants to improvide an existing feature label Mar 9, 2020
@Yukaii Yukaii temporarily deployed to codimd-feature-pdf-expo-las27r March 9, 2020 13:41 Inactive
@Yukaii Yukaii temporarily deployed to codimd-feature-pdf-expo-las27r March 9, 2020 13:44 Inactive
@Yukaii Yukaii had a problem deploying to codimd-feature-pdf-expo-las27r March 9, 2020 13:50 Failure
@Yukaii Yukaii had a problem deploying to codimd-feature-pdf-expo-las27r March 14, 2020 15:32 Failure
@Yukaii
Copy link
Member Author

Yukaii commented Mar 14, 2020

Heroku image is too big when building with puppeteer buildpack

I'll try to move all webpack build dependency into devDependency, then let heroku remove them automatically.

@Yukaii
Copy link
Member Author

Yukaii commented Mar 14, 2020

-----> Compressing...
 !     Compiled slug size: 612.6M is too large (max is 500M).
 !     See: http://devcenter.heroku.com/articles/slug-size
 !     Push failed

https://devcenter.heroku.com/articles/slug-compiler#slug-size

@Yukaii Yukaii had a problem deploying to codimd-feature-pdf-expo-las27r March 19, 2020 12:43 Failure
@Yukaii Yukaii had a problem deploying to codimd-feature-pdf-expo-las27r March 19, 2020 13:00 Failure
@Yukaii
Copy link
Member Author

Yukaii commented Mar 19, 2020

-----> Compressing...
 !     Compiled slug size: 520.4M is too large (max is 500M).
 !     See: http://devcenter.heroku.com/articles/slug-size
 !     Push failed

well, well 😂

Signed-off-by: Yukai Huang <[email protected]>
@Yukaii Yukaii temporarily deployed to codimd-feature-pdf-expo-las27r March 19, 2020 13:22 Inactive
@Yukaii
Copy link
Member Author

Yukaii commented Mar 19, 2020

       Procfile declares types -> web
-----> Compressing...
       Done: 493.6M
-----> Launching...
 !     Warning: Your slug size (493 MB) exceeds our soft limit (300 MB) which may affect boot time.
       Released v10
       https://codimd-feature-pdf-expo-las27r.herokuapp.com/ deployed to Heroku

@jackycute jackycute removed this from the Next milestone May 12, 2021
edgarogh pushed a commit to WartaPoirier-corp/codimd that referenced this pull request Sep 21, 2021
chore(deps): update dependency eslint to v7.31.0 (master)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Wants to improvide an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants