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.

Sign up for GitHub

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

Jump to bottom

getCloudflareContext async #371

Closed
vicb opened this issue Feb 12, 2025 · 1 comment · Fixed by #372
Closed

getCloudflareContext async #371

vicb opened this issue Feb 12, 2025 · 1 comment · Fixed by #372
Assignees
@dario-piotrowicz

Comments

@vicb
Copy link
Contributor

vicb commented Feb 12, 2025 •
edited
Loading

That would be helpful for SSG

But might get local data (D1) at build time.

Needs to investigate
@vicb vicb converted this from a draft issue Feb 12, 2025
@dario-piotrowicz
Copy link
Contributor

But might get local data (D1) at build time.
Needs to investigate

As we discussed this is probably not a big deal, I think we can just proceed with this and then make sure to properly document these aspects in the docs (potentially in the dev/getCloudflareContext how-to guide) 🙂

@dario-piotrowicz dario-piotrowicz mentioned this issue Feb 12, 2025
Merged
@vicb vicb closed this as completed in #372 Feb 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dario-piotrowicz dario-piotrowicz

Labels
None yet
Projects
`@opennextjs/cloudflare` prioritization and tracking
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

add "async mode" to getCloudflareContext opennextjs/opennextjs-cloudflare
2 participants
@vicb @dario-piotrowicz