Skip to content

Populating cache should warn when binding is not present #530

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

Closed
vicb opened this issue Apr 4, 2025 · 2 comments
Closed

Populating cache should warn when binding is not present #530

vicb opened this issue Apr 4, 2025 · 2 comments

Comments

@vicb
Copy link
Contributor

vicb commented Apr 4, 2025

wrangler.jsonc

	"r2_buckets": [],

open-next.config.ts

import { defineCloudflareConfig } from "@opennextjs/cloudflare";
import r2IncrementalCache from "@opennextjs/cloudflare/overrides/incremental-cache/r2-incremental-cache";

export default defineCloudflareConfig({
  incrementalCache: r2IncrementalCache,
});

populateCache:

-> % pnpm opennextjs-cloudflare populateCache local  

┌─────────────────────────────────────┐
│ OpenNext — Cloudflare populateCache │
└─────────────────────────────────────┘


Populating R2 incremental cache...
Successfully populated cache with 4 assets
Tag cache does not need populating
@james-elicx
Copy link
Collaborator

Wondering if it would be better to have Wrangler serve errors if a local binding is not configured in a wrangler.json?

@vicb vicb changed the title Populating cache does should warn when binding is not present Populating cache should warn when binding is not present Apr 4, 2025
@vicb
Copy link
Contributor Author

vicb commented Apr 9, 2025

fixed via #546 and #536

@vicb vicb closed this as completed Apr 9, 2025
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

No branches or pull requests

2 participants