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.

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

Update stackblitz embed src from /simple → /fixed #182

Merged
merged 2 commits into from
Feb 12, 2024

Conversation

acusti
Copy link
Contributor

@acusti acusti commented Feb 9, 2024

fixes #174

  • examples/${framework}/simple doesn’t exist in any of the react, svelte, or vue examples folders
  • examples/${framework}/fixed does exist in all of the framework example folders and seems like the equivalent to “simple”

examples/react/simple doesn’t exist
examples/react/fixed does exist and seems like the equivalent to “simple”
Copy link

vercel bot commented Feb 9, 2024

@acusti is attempting to deploy a commit to the Tanstack Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Contributor

@fulopkovacs fulopkovacs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix! ☺️ Could you fix the same issue for Ranger too? The StackBlitz project should point at the basic example (https://github.com/TanStack/ranger/tree/main/examples/react/basic) instead of simple (which does not exist).

same fix as be3b03d applied to the ranger library
@acusti
Copy link
Contributor Author

acusti commented Feb 12, 2024

@fulopkovacs thanks for the quick review! i updated the PR as requested.

Copy link
Contributor

@fulopkovacs fulopkovacs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, thanks for the contribution! 🚀

@fulopkovacs fulopkovacs merged commit 8fc674a into TanStack:main Feb 12, 2024
2 of 3 checks passed
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

Successfully merging this pull request may close these issues.

preview broken at tanstack.com/virtual
2 participants