docs(guides/envvars): clarify docs for local/production env variables

[dgurns]

When I recently spun up a project using the @remix-run/cloudflare-pages adapter, it was not clear how to set up and use env variables, both locally and when deployed to Cloudflare Pages.

Thankfully @jemjam figured it out in this issue comment

This updates the Environment Variables docs with that information. This should hopefully save people a lot of pain when initially setting up a new project with the Cloudflare Pages adapter.

[machour]

@dgurns Thank you for opening this PR !

This seems a little too specific to me, and perhaps should be added in a new, CF specific, section. @kentcdodds WDYT?

[jemjam]

aha, I didn't even think to cat the env file, I've been binding these values individually (like a fool)

[dgurns]

I gotta give a h/t to @theeomm on that one! #3186 (comment)

[jemjam]

Just threw some minor thoughts up, take or leave my opinions if you choose to use any of this. ❤️

[remix-cla-bot]

Thank you for signing the Contributor License Agreement. Let's get this merged! 🥳

[dgurns]

Thanks for the great feedback folks. I updated the doc to be better organized, dividing "Server Environment Variables" into "Local Development" and "Production" sections.

[MichaelDeBoey]

@theeomm already updated the cloudflare-pages template in #3186, so I don't think these doc changes are necessary anymore once his PR is merged? 🤔

[dgurns]

@theeomm already updated the cloudflare-pages template in #3186, so I don't think these doc changes are necessary anymore once his PR is merged? 🤔

That PR looks great and will help, but I think this one is important as well. If you're a new user with a Cloudflare Pages project:

• It's not clear that you can use a .env file - usually Cloudflare projects use a wrangler.toml locally for env variables, if I understand correctly.
• Once you have the env variables loaded in, it's not clear how to access them. I initially reached for process.env by default, and then top-level global variables like Cloudflare Workers, before realizing by trial and error that they were on context.

[theeomm]

@theeomm already updated the cloudflare-pages template in #3186, so I don't think these doc changes are necessary anymore once his PR is merged? 🤔

That PR looks great and will help, but I think this one is important as well. If you're a new user with a Cloudflare Pages project:

• It's not clear that you can use a .env file - usually Cloudflare projects use a wrangler.toml locally for env variables, if I understand correctly.
• Once you have the env variables loaded in, it's not clear how to access them. I initially reached for process.env by default, and then top-level global variables like Cloudflare Workers, before realizing by trial and error that they were on context.

I think it is necessary for this PR to be merged too as it really documents how to handle environment variables with CF, which I should have probably done in my PR in the first place.

[kentcdodds]

Thanks everyone!

[dgurns]

Cool! Gotta say I am super impressed with the Remix community so far. Thank you all for jumping in with feedback so quickly on the PR, and seeing it merged in a few days. Amazing stuff! Excited to keep digging in to Remix.