jbmoelker commented 11 months ago

Changes

Adds support for dynamic API routes on Cloudflare (using @astro/cloudflare to enable Cloudflare Workers).
Adds access to Cloudflare runtime object in pages (Astro.locals.runtime) and API routes (context.locals.runtime).
Extends local preview setup with Cloudflare runtime support.
Upgrades from Astro v2 to Astro v3 since it's stable and introduces a new API route syntax.
Adds demo endpoints
Redirects root url to home page in user's preferred language. Uses a dynamic endpoint in the page root and parses the language accept header.

Associated issue

Open preview link
Verify you are redirected to home page in your preferred (browser/system) locale (or to the default locale if yours isn't available).
Test API endpoint using URL searchParams: /api/demo/hello?to=you
Verify it responds with dynamic JSON (you can change the to value)
Test API endpoint using request.cf props: /api/demo/geo
Verify it responds with dynamic JSON (your location)

[x] I have performed a self-review of my own code
[x] I have made sure that my PR is easy to review (not too big, includes comments)
~~I have added/updated tests to prove that my feature works (if not possible please explain why)~~
[x] I have made changes to the README and if the change affects the project setup (npm commands changed, new service added, environmental variable added)
~~I have added a decision log entry if the change affects the architecture or changes a significant technology~~
[x] I have notified a reviewer

decrek commented 11 months ago

With the new icon implementation there are no problems anymore right? So indeed I would add wrangler for development and the rest looks good.

cloudflare-workers-and-pages[bot] commented 11 months ago

Latest commit:	`c2e0f6b`
Status:	✅ Deploy successful!
Preview URL:	https://869fc4e7.head-start.pages.dev
Branch Preview URL:	https://feat-api-routes.head-start.pages.dev

jbmoelker commented 11 months ago

With the new icon implementation there are no problems anymore right? So indeed I would add wrangler for development and the rest looks good.