QwikDev / qwik

Instant-loading web apps, without effort
https://qwik.dev
MIT License
20.85k stars 1.31k forks source link

docs: fix useResource example & remove unused demo #6893

Closed ianlet closed 4 weeks ago

ianlet commented 1 month ago

What is it?

Description

Fixes https://github.com/orgs/QwikDev/projects/1/views/2?pane=issue&itemId=72272653

The previous useResource example was using the github api, which might get rate limited. Instead, the example now uses a free fake and stable API. Potential errors are also caught in order to avoid other issues.

In addition, this PR deletes the unused demo demo/tasks/resource

Note: the error mentioned in the issue doesn't actually happen in local. So this is a bit of a wild guess as to why it might fail in production.

Checklist:

changeset-bot[bot] commented 1 month ago

🦋 Changeset detected

Latest commit: 278695ad45ad7bc488c6245b3e15e5b8b0c4a49b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages | Name | Type | | --------------------- | ----- | | @builder.io/qwik | Patch | | eslint-plugin-qwik | Patch | | @builder.io/qwik-city | Patch | | create-qwik | Patch |

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

github-actions[bot] commented 1 month ago
built with Refined Cloudflare Pages Action

âš¡ Cloudflare Pages Deployment

Name Status Preview Last Commit
qwik-docs ✅ Ready (View Log) Visit Preview 278695ad45ad7bc488c6245b3e15e5b8b0c4a49b