-
Notifications
You must be signed in to change notification settings - Fork 73
fix: remove dynamic require for map file #347
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
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
🦋 Changeset detectedLatest commit: cb75acf The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
dario-piotrowicz
approved these changes
Feb 6, 2025
Thanks Dario! |
wesbos
added a commit
to wesbos/wesbos
that referenced
this pull request
Feb 6, 2025
wesbos
added a commit
to wesbos/wesbos
that referenced
this pull request
May 12, 2025
* adds cloudflare puppeteer support * fix broken CSS * try remove dynamic import * add caching * disable hls video element as its too large for cloudflare workers.. for now * bundle analyzer * try custom route for workers * close browser after use * add logging * close browser on caught error * upgrade opennextjs to fix opennextjs/opennextjs-cloudflare#234 * add public folder to repo. * dynamic import of puppeteer * logging and import fix * arg * asdfasdf * migrate puppeteer to use local chrome instead * conditinal import * adds cobalt2 theme for highlighting. Fixes #407 * WIP metadata * blog post meta tags * Image loader for cloudflare * custom omain * fallback quality * fix footer styling. closes #413 * worker + jpg * smaller * jpg * favicon formatting * square favicon * Stretch video. fixes #418 * Meta WIP * Meta WIP * Page meta WIP * js page to mdx * Move Shiki to singleton. Fixes #423 * Page Meta complete. I hope * fixes leaking global CSS for OG images * Meta tag finished * fix the types * cleanup * Sentry integreation * reinstate content navigation. Fix post cache * content Nav and page meta fixed * fix build * minor deps * migrate to initOpenNextCloudflareForDev. Make getCloudflareContext sync * adds twitter feed back in * Reinstates welcome image. Closes #428 * match nav * logs * fix images on localhost * not found * enable PPR and dynamic tips page * Upgrade opennextjs to fix dynamic require bug. opennextjs/opennextjs-cloudflare#347 * remove build skip * remove global cache to see if memory improves * disable instrumentation * try PR #354 to fix memory issues * reinstate instrumentation * reinstate global cache * global error page * remove patches * remove tunnel route * fix lockfile * reinstate patches * remove last bits of gatsby code * hRM * disable async metadata call * add workers domain to wrangler file * NVM * remove ppr * upgrade next canary * disable PPR * remove next import * revert to stable next * reinstate PPR * remove headers call from not-found. Causing RenderHTML Bug opennextjs/opennextjs-cloudflare#333 * Adds static gen for all blog posts and javascript notes pages * image compressor script * compresor script * compress images * adds minifiy to wrangler deploy * removes unused images * fake change to trigger cf * reinstate footer * PPR for footer blocks * revert next.js canary version * next stable * experimental ppr * force dynamic everyhting * make all pages dyanmic * remove old templates * toggle instrumentation * upgrade deps * override axios dependency to add fetch support * Creates MDX static export file * Last few commits before I switch to waku * move to waku * waku hono cache * Caching middleware * dynamic LAyout * dynamic contact page * disable cache * Implement caching wrapper with KV * Adds cache * PAge meta * clean up cache * image sizes are working! * unpic image component * eager * css codesplit * nice fontstack fallback * Smaller logo * add preload for fonts * add preload for fonts * inline source * revert * deps * codesplit * black grit is now a webp * preload webfonts * remove css codesplit * handsome preload * preload handsome * add width/height to footer image * font-display: swap; * accessibility * hard code logo size * asset caching * remove client provider as it was leaking server component into client * adds font fallback with proper sizing * adds logging for cache * cache RSC requests, plus about and contact pages * favicon * CDN-Cache-Control * favicon file * adds cache time header * Ugh * increase cache time * remove public * LCP home image * try larger image * disable grit * page speed hacks * put the grit back in * cachce * revert * upgrade image-size to 2.0 * cache * hrm * cache * x-cache-hit * asfd * cache * asdfasdf * cache * cache * cache * cache * return cache works, testing fix for initial work * tee body * tee body * no dupe * tee * change to CDN cache control * change to CDN cache control * adds cache tag and client TTL headers * adds git hash to cache key and cache tag * clean things up * OG images * home page og * removes next.js app directory * new nav * robots.txt * pagination * pagination title tags * git hash * error boundary testing * move break component * asdf * hono middleware * prod * order * tails * remove error * Fix types for css modules * adds social links. Fixes #427 * Static paths for blog posts. Fixes #436 * custom 404 * debugging x requests * Custom 404 page shenangins * env variables * dev + deploy env * fix image size inlining * REinstates page scrolling for JS notes. fixes #410 * add sounds on hover * sounds * new syntax artwork * Implement Biome as linter and formatter * formatting * Linting fixes * social links * error handling integration * mobile nav * removes caching onf 404, 403 and 500 pages * Update 50-mapping.mdx Fixed some of the capitalization for toUpperCase() function, the parentheses for order of operations in the age calculations, and the attribute in the person object. * Update 57-shopping-form-with-custom-events-delegation-and-localstorage.mdx Make sure that first argument sent to localStorage.setItem is a string. * Update 21-selecting-elements.mdx * fix syntax links * Removes border inline when kissing * Fixes edit links * removes error testing buttons * Upgrade to latest deps, migrate to waku string config * Page loading indicator * fix javascript notes scroling issues * LETS launch this sucker * upgrade deps * fix the build * Sourcemaps * Trying some new error logging * Trying some new error logging * logging * logging * reinstate create tip CLI * patch mediachrome to work on workers * analytics * Analytics logging * client component * readme * custom domain * sub * traiing slash redirect * error logging * add domain to traceProp * logging * try remove explicit reporting. I think the error boundary is catching * Migrate to router events API. Fixes #455 * upgrade wrangler * update wrangler * add github mcp * Update 50-mapping.mdx Fixed some of the capitalization for toUpperCase() function, the parentheses for order of operations in the age calculations, and the attribute in the person object. --------- Co-authored-by: Wes Bos <[email protected]> Co-authored-by: upendradhami <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
fixes #341
ESBuild tries to load all the files in
/chunks
when it seesrequire("./chunks/" + var)
.If that folder contains map file then it will error with
No loader is configured for ".map" files
The fix is to remove that unused
require("./chunks/" + var)