-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
fix(useLoader): dispose loaders #2984
Conversation
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit c388024:
|
This reverts commit 83ae5d1.
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [@react-three/fiber](https://togithub.com/pmndrs/react-three-fiber) | [`8.13.0` -> `8.14.4`](https://renovatebot.com/diffs/npm/@react-three%2ffiber/8.13.0/8.14.4) | [![age](https://developer.mend.io/api/mc/badges/age/npm/@react-three%2ffiber/8.14.4?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/@react-three%2ffiber/8.14.4?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/@react-three%2ffiber/8.13.0/8.14.4?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/@react-three%2ffiber/8.13.0/8.14.4?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>pmndrs/react-three-fiber (@​react-three/fiber)</summary> ### [`v8.14.4`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.14.4) [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/v8.14.3...v8.14.4) #### What's Changed - fix(native): amend BlobManager over globals by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/3030](https://togithub.com/pmndrs/react-three-fiber/pull/3030) #### New Contributors - [@​lin-stephanie](https://togithub.com/lin-stephanie) made their first contribution in [https://github.com/pmndrs/react-three-fiber/pull/3024](https://togithub.com/pmndrs/react-three-fiber/pull/3024) **Full Changelog**: pmndrs/react-three-fiber@v8.14.3...v8.14.4 ### [`v8.14.3`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.14.3) [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/v8.14.2...v8.14.3) #### What's Changed - fix(native): drop fsstat for react-native-web by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/3023](https://togithub.com/pmndrs/react-three-fiber/pull/3023) **Full Changelog**: pmndrs/react-three-fiber@v8.14.2...v8.14.3 ### [`v8.14.2`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.14.2) [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/v8.14.1...v8.14.2) #### What's Changed - perf(native): don't double encode assets by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/2996](https://togithub.com/pmndrs/react-three-fiber/pull/2996) - fix(useLoader): loosen Loader onError signature by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/3011](https://togithub.com/pmndrs/react-three-fiber/pull/3011) **Full Changelog**: pmndrs/react-three-fiber@v8.14.1...v8.14.2 ### [`v8.14.1`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.14.1) [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/v8.14.0...v8.14.1) #### What's Changed - fix(native): prefer local uri for fs by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/2993](https://togithub.com/pmndrs/react-three-fiber/pull/2993) **Full Changelog**: pmndrs/react-three-fiber@v8.14.0...v8.14.1 ### [`v8.14.0`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.14.0): 8.14.0 [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/v8.13.9...v8.14.0) ##### What's Changed - experiment(native): implement missing Blob/URL/fetch APIs by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/2982](https://togithub.com/pmndrs/react-three-fiber/pull/2982) - feat(native): use PanResponder for react-native-web by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/2985](https://togithub.com/pmndrs/react-three-fiber/pull/2985) **Full Changelog**: pmndrs/react-three-fiber@v8.13.9...v8.14.0 ### [`v8.13.9`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.13.9) [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/v8.13.8...v8.13.9) #### What's Changed - fix(native): TextureLoader should remain consistent with FileLoader by [@​l3utterfly](https://togithub.com/l3utterfly) in [https://github.com/pmndrs/react-three-fiber/pull/2986](https://togithub.com/pmndrs/react-three-fiber/pull/2986) #### New Contributors - [@​l3utterfly](https://togithub.com/l3utterfly) made their first contribution in [https://github.com/pmndrs/react-three-fiber/pull/2986](https://togithub.com/pmndrs/react-three-fiber/pull/2986) **Full Changelog**: pmndrs/react-three-fiber@v8.13.8...v8.13.9 ### [`v8.13.8`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.13.8) [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/8.13.7...v8.13.8) #### What's Changed - fix(useLoader): dispose loaders by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/2984](https://togithub.com/pmndrs/react-three-fiber/pull/2984) #### New Contributors - [@​GmBodhi](https://togithub.com/GmBodhi) made their first contribution in [https://github.com/pmndrs/react-three-fiber/pull/2976](https://togithub.com/pmndrs/react-three-fiber/pull/2976) **Full Changelog**: pmndrs/react-three-fiber@8.13.7...v8.13.8 ### [`v8.13.7`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/8.13.7) [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/v8.13.6...8.13.7) #### What's Changed - fix(Canvas): pass scene prop by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/2975](https://togithub.com/pmndrs/react-three-fiber/pull/2975) #### New Contributors - [@​GabeDahl](https://togithub.com/GabeDahl) made their first contribution in [https://github.com/pmndrs/react-three-fiber/pull/2968](https://togithub.com/pmndrs/react-three-fiber/pull/2968) **Full Changelog**: pmndrs/react-three-fiber@v8.13.6...8.13.7 ### [`v8.13.6`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.13.6) [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/v8.13.5...v8.13.6) #### What's Changed - fix: harden XR init against Renderer shim by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/2947](https://togithub.com/pmndrs/react-three-fiber/pull/2947) #### New Contributors - [@​hood](https://togithub.com/hood) made their first contribution in [https://github.com/pmndrs/react-three-fiber/pull/2898](https://togithub.com/pmndrs/react-three-fiber/pull/2898) - [@​Sargsian](https://togithub.com/Sargsian) made their first contribution in [https://github.com/pmndrs/react-three-fiber/pull/2922](https://togithub.com/pmndrs/react-three-fiber/pull/2922) - [@​myznikovgleb](https://togithub.com/myznikovgleb) made their first contribution in [https://github.com/pmndrs/react-three-fiber/pull/2935](https://togithub.com/pmndrs/react-three-fiber/pull/2935) **Full Changelog**: pmndrs/react-three-fiber@v8.13.5...v8.13.6 ### [`v8.13.5`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.13.5) [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/v8.13.4...v8.13.5) #### What's Changed - fix(types): remove reference to [@​types/webxr](https://togithub.com/types/webxr) by [@​Methuselah96](https://togithub.com/Methuselah96) in [https://github.com/pmndrs/react-three-fiber/pull/2919](https://togithub.com/pmndrs/react-three-fiber/pull/2919) - fix(types): harden colors overload since r153 by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/2931](https://togithub.com/pmndrs/react-three-fiber/pull/2931) **Full Changelog**: pmndrs/react-three-fiber@v8.13.4...v8.13.5 ### [`v8.13.4`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.13.4) [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/@react-three/[email protected]) ##### What's Changed - fix: safely diff instances by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/2906](https://togithub.com/pmndrs/react-three-fiber/pull/2906) ##### New Contributors - [@​bhouston](https://togithub.com/bhouston) made their first contribution in [https://github.com/pmndrs/react-three-fiber/pull/2896](https://togithub.com/pmndrs/react-three-fiber/pull/2896) **Full Changelog**: pmndrs/react-three-fiber@v8.13.2...v8.13.4 ### [`v8.13.3`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.13.3) [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/@react-three/[email protected]...@react-three/[email protected]) #### What's Changed - fix: revert nestable portals, up suspend-react by [@​drcmda](https://togithub.com/drcmda) in [`05187c2`](https://togithub.com/pmndrs/react-three-fiber/commit/05187c22fcac8d480120ffe1133ebe85049cb552) **Full Changelog**: pmndrs/react-three-fiber@v8.13.2...v8.13.3 ### [`v8.13.2`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.13.2) [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/v8.13.1...@react-three/[email protected]) #### What's Changed - fix: update suspend-react by [@​drcmda](https://togithub.com/drcmda) in [`95e71ae`](https://togithub.com/pmndrs/react-three-fiber/commit/95e71ae00e8ede868b9584891e8e9b04d54d0f4e) **Full Changelog**: pmndrs/react-three-fiber@v8.13.1...v8.13.2 ### [`v8.13.1`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.13.1) [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/v8.13.0...v8.13.1) #### What's Changed - docs: update links to legacy react docs by [@​ninofiliu](https://togithub.com/ninofiliu) in [https://github.com/pmndrs/react-three-fiber/pull/2858](https://togithub.com/pmndrs/react-three-fiber/pull/2858) - fix(core): unlink primitives on dispose by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/2877](https://togithub.com/pmndrs/react-three-fiber/pull/2877) - fix(core): portals should be nestable by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/2746](https://togithub.com/pmndrs/react-three-fiber/pull/2746) #### New Contributors - [@​ninofiliu](https://togithub.com/ninofiliu) made their first contribution in [https://github.com/pmndrs/react-three-fiber/pull/2858](https://togithub.com/pmndrs/react-three-fiber/pull/2858) **Full Changelog**: pmndrs/react-three-fiber@v8.13.0...v8.13.1 </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Enabled. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/ziyadedher/ziyadedher). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4wLjMiLCJ1cGRhdGVkSW5WZXIiOiIzNy4wLjMiLCJ0YXJnZXRCcmFuY2giOiJtYWluIn0=--> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [@react-three/fiber](https://togithub.com/pmndrs/react-three-fiber) | [`8.13.0` -> `8.14.4`](https://renovatebot.com/diffs/npm/@react-three%2ffiber/8.13.0/8.14.4) | [![age](https://developer.mend.io/api/mc/badges/age/npm/@react-three%2ffiber/8.14.4?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/@react-three%2ffiber/8.14.4?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/@react-three%2ffiber/8.13.0/8.14.4?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/@react-three%2ffiber/8.13.0/8.14.4?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>pmndrs/react-three-fiber (@​react-three/fiber)</summary> ### [`v8.14.4`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.14.4) [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/v8.14.3...v8.14.4) #### What's Changed - fix(native): amend BlobManager over globals by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/3030](https://togithub.com/pmndrs/react-three-fiber/pull/3030) #### New Contributors - [@​lin-stephanie](https://togithub.com/lin-stephanie) made their first contribution in [https://github.com/pmndrs/react-three-fiber/pull/3024](https://togithub.com/pmndrs/react-three-fiber/pull/3024) **Full Changelog**: pmndrs/react-three-fiber@v8.14.3...v8.14.4 ### [`v8.14.3`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.14.3) [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/v8.14.2...v8.14.3) #### What's Changed - fix(native): drop fsstat for react-native-web by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/3023](https://togithub.com/pmndrs/react-three-fiber/pull/3023) **Full Changelog**: pmndrs/react-three-fiber@v8.14.2...v8.14.3 ### [`v8.14.2`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.14.2) [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/v8.14.1...v8.14.2) #### What's Changed - perf(native): don't double encode assets by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/2996](https://togithub.com/pmndrs/react-three-fiber/pull/2996) - fix(useLoader): loosen Loader onError signature by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/3011](https://togithub.com/pmndrs/react-three-fiber/pull/3011) **Full Changelog**: pmndrs/react-three-fiber@v8.14.1...v8.14.2 ### [`v8.14.1`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.14.1) [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/v8.14.0...v8.14.1) #### What's Changed - fix(native): prefer local uri for fs by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/2993](https://togithub.com/pmndrs/react-three-fiber/pull/2993) **Full Changelog**: pmndrs/react-three-fiber@v8.14.0...v8.14.1 ### [`v8.14.0`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.14.0): 8.14.0 [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/v8.13.9...v8.14.0) ##### What's Changed - experiment(native): implement missing Blob/URL/fetch APIs by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/2982](https://togithub.com/pmndrs/react-three-fiber/pull/2982) - feat(native): use PanResponder for react-native-web by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/2985](https://togithub.com/pmndrs/react-three-fiber/pull/2985) **Full Changelog**: pmndrs/react-three-fiber@v8.13.9...v8.14.0 ### [`v8.13.9`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.13.9) [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/v8.13.8...v8.13.9) #### What's Changed - fix(native): TextureLoader should remain consistent with FileLoader by [@​l3utterfly](https://togithub.com/l3utterfly) in [https://github.com/pmndrs/react-three-fiber/pull/2986](https://togithub.com/pmndrs/react-three-fiber/pull/2986) #### New Contributors - [@​l3utterfly](https://togithub.com/l3utterfly) made their first contribution in [https://github.com/pmndrs/react-three-fiber/pull/2986](https://togithub.com/pmndrs/react-three-fiber/pull/2986) **Full Changelog**: pmndrs/react-three-fiber@v8.13.8...v8.13.9 ### [`v8.13.8`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.13.8) [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/8.13.7...v8.13.8) #### What's Changed - fix(useLoader): dispose loaders by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/2984](https://togithub.com/pmndrs/react-three-fiber/pull/2984) #### New Contributors - [@​GmBodhi](https://togithub.com/GmBodhi) made their first contribution in [https://github.com/pmndrs/react-three-fiber/pull/2976](https://togithub.com/pmndrs/react-three-fiber/pull/2976) **Full Changelog**: pmndrs/react-three-fiber@8.13.7...v8.13.8 ### [`v8.13.7`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/8.13.7) [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/v8.13.6...8.13.7) #### What's Changed - fix(Canvas): pass scene prop by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/2975](https://togithub.com/pmndrs/react-three-fiber/pull/2975) #### New Contributors - [@​GabeDahl](https://togithub.com/GabeDahl) made their first contribution in [https://github.com/pmndrs/react-three-fiber/pull/2968](https://togithub.com/pmndrs/react-three-fiber/pull/2968) **Full Changelog**: pmndrs/react-three-fiber@v8.13.6...8.13.7 ### [`v8.13.6`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.13.6) [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/v8.13.5...v8.13.6) #### What's Changed - fix: harden XR init against Renderer shim by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/2947](https://togithub.com/pmndrs/react-three-fiber/pull/2947) #### New Contributors - [@​hood](https://togithub.com/hood) made their first contribution in [https://github.com/pmndrs/react-three-fiber/pull/2898](https://togithub.com/pmndrs/react-three-fiber/pull/2898) - [@​Sargsian](https://togithub.com/Sargsian) made their first contribution in [https://github.com/pmndrs/react-three-fiber/pull/2922](https://togithub.com/pmndrs/react-three-fiber/pull/2922) - [@​myznikovgleb](https://togithub.com/myznikovgleb) made their first contribution in [https://github.com/pmndrs/react-three-fiber/pull/2935](https://togithub.com/pmndrs/react-three-fiber/pull/2935) **Full Changelog**: pmndrs/react-three-fiber@v8.13.5...v8.13.6 ### [`v8.13.5`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.13.5) [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/v8.13.4...v8.13.5) #### What's Changed - fix(types): remove reference to [@​types/webxr](https://togithub.com/types/webxr) by [@​Methuselah96](https://togithub.com/Methuselah96) in [https://github.com/pmndrs/react-three-fiber/pull/2919](https://togithub.com/pmndrs/react-three-fiber/pull/2919) - fix(types): harden colors overload since r153 by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/2931](https://togithub.com/pmndrs/react-three-fiber/pull/2931) **Full Changelog**: pmndrs/react-three-fiber@v8.13.4...v8.13.5 ### [`v8.13.4`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.13.4) [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/@react-three/[email protected]) ##### What's Changed - fix: safely diff instances by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/2906](https://togithub.com/pmndrs/react-three-fiber/pull/2906) ##### New Contributors - [@​bhouston](https://togithub.com/bhouston) made their first contribution in [https://github.com/pmndrs/react-three-fiber/pull/2896](https://togithub.com/pmndrs/react-three-fiber/pull/2896) **Full Changelog**: pmndrs/react-three-fiber@v8.13.2...v8.13.4 ### [`v8.13.3`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.13.3) [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/@react-three/[email protected]...@react-three/[email protected]) #### What's Changed - fix: revert nestable portals, up suspend-react by [@​drcmda](https://togithub.com/drcmda) in [`05187c2`](https://togithub.com/pmndrs/react-three-fiber/commit/05187c22fcac8d480120ffe1133ebe85049cb552) **Full Changelog**: pmndrs/react-three-fiber@v8.13.2...v8.13.3 ### [`v8.13.2`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.13.2) [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/v8.13.1...@react-three/[email protected]) #### What's Changed - fix: update suspend-react by [@​drcmda](https://togithub.com/drcmda) in [`95e71ae`](https://togithub.com/pmndrs/react-three-fiber/commit/95e71ae00e8ede868b9584891e8e9b04d54d0f4e) **Full Changelog**: pmndrs/react-three-fiber@v8.13.1...v8.13.2 ### [`v8.13.1`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.13.1) [Compare Source](https://togithub.com/pmndrs/react-three-fiber/compare/v8.13.0...v8.13.1) #### What's Changed - docs: update links to legacy react docs by [@​ninofiliu](https://togithub.com/ninofiliu) in [https://github.com/pmndrs/react-three-fiber/pull/2858](https://togithub.com/pmndrs/react-three-fiber/pull/2858) - fix(core): unlink primitives on dispose by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/2877](https://togithub.com/pmndrs/react-three-fiber/pull/2877) - fix(core): portals should be nestable by [@​CodyJasonBennett](https://togithub.com/CodyJasonBennett) in [https://github.com/pmndrs/react-three-fiber/pull/2746](https://togithub.com/pmndrs/react-three-fiber/pull/2746) #### New Contributors - [@​ninofiliu](https://togithub.com/ninofiliu) made their first contribution in [https://github.com/pmndrs/react-three-fiber/pull/2858](https://togithub.com/pmndrs/react-three-fiber/pull/2858) **Full Changelog**: pmndrs/react-three-fiber@v8.13.0...v8.13.1 </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Enabled. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/ziyadedher/ziyadedher). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4wLjMiLCJ1cGRhdGVkSW5WZXIiOiIzNy4wLjMiLCJ0YXJnZXRCcmFuY2giOiJtYWluIn0=--> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@@ -103,7 +110,7 @@ function loadingFn<L extends LoaderProto<any>>( | |||
), | |||
), | |||
), | |||
) | |||
).finally(() => (loader as any).dispose?.()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hey @CodyJasonBennett this breaks using DRACOLoader
directly (useLoader(DRACOLoader, url)
, without useGTLF()
or similar), since DRACOLoader
cannot be used again after dispose https://threejs.org/docs/#examples/en/loaders/DRACOLoader.dispose
At the moment I simply workaround this by making dispose()
not do anything.
While debugging this issue I also checked #3131, and I think it won't work as expected, I think dispose()
should not be called if the user wants to reuse a particular loader.
What do you think?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I agree. This was an oversight.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixes #2812.