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

Remove ISR as capture-website is not working on serverless functions #75

Merged
merged 6 commits into from Jun 2, 2022

Conversation

joulev
Copy link
Owner

@joulev joulev commented Jun 2, 2022

Instead I will use swr with client-side fetching for my project log. This will close #74.

And now instead of the project log page only getting updated once a day, it will be updated live, with full fallback data.

Since getOgImage will only be run during build time now, it should be fine.

@joulev joulev added the frontend All issues related to the front-end label Jun 2, 2022
@joulev

This comment was marked as resolved.

@codecov
Copy link

codecov bot commented Jun 2, 2022

Codecov Report

Merging #75 (c76ab96) into main (5b39b0f) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #75      +/-   ##
==========================================
- Coverage   98.75%   98.75%   -0.01%     
==========================================
  Files          30       30              
  Lines         402      401       -1     
  Branches       96       95       -1     
==========================================
- Hits          397      396       -1     
  Misses          5        5              
Impacted Files Coverage Δ
packages/client/lib/getOgImage.ts 100.00% <100.00%> (ø)

Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
@joulev
Copy link
Owner Author

joulev commented Jun 2, 2022

Good to go.

@joulev joulev merged commit bfa4854 into main Jun 2, 2022
@joulev joulev deleted the puppeteer-aws-error-fix branch June 2, 2022 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend All issues related to the front-end
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Capturing og:image with Puppeteer throws errors
1 participant