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

fix(applyProps): null check indeterminate instances #3261

Merged
merged 2 commits into from
May 10, 2024

Conversation

CodyJasonBennett
Copy link
Member

Fixes an issue reported in Discord where events add/remove in applyProps would write to an empty object.

Copy link

codesandbox-ci bot commented May 9, 2024

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 1b39f92:

Sandbox Source
example Configuration

@cyango
Copy link

cyango commented May 10, 2024

Eagerly waiting for this :)

@CodyJasonBennett CodyJasonBennett merged commit 72ace15 into master May 10, 2024
2 checks passed
@CodyJasonBennett CodyJasonBennett deleted the fix/null-check-indeterminate-instances branch May 10, 2024 17:59
@cyango
Copy link

cyango commented May 10, 2024

After updating, now this happens for example when i resize the window:
image

r4ai referenced this pull request in r4ai/r4ai.dev May 15, 2024
[![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://github.com/pmndrs/react-three-fiber) |
[`8.16.3` ->
`8.16.6`](https://renovatebot.com/diffs/npm/@react-three%2ffiber/8.16.3/8.16.6)
|
[![age](https://developer.mend.io/api/mc/badges/age/npm/@react-three%2ffiber/8.16.6?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/@react-three%2ffiber/8.16.6?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/@react-three%2ffiber/8.16.3/8.16.6?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/@react-three%2ffiber/8.16.3/8.16.6?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>pmndrs/react-three-fiber (@&#8203;react-three/fiber)</summary>

###
[`v8.16.6`](https://github.com/pmndrs/react-three-fiber/releases/tag/v8.16.6)

[Compare
Source](https://github.com/pmndrs/react-three-fiber/compare/v8.16.5...v8.16.6)

#### What's Changed

- fix(applyProps): null check indeterminate instances by
[@&#8203;CodyJasonBennett](https://github.com/CodyJasonBennett) in
[https://github.com/pmndrs/react-three-fiber/pull/3261](https://github.com/pmndrs/react-three-fiber/pull/3261)

**Full Changelog**:
pmndrs/react-three-fiber@v8.16.5...v8.16.6

###
[`v8.16.5`](https://github.com/pmndrs/react-three-fiber/releases/tag/v8.16.5)

[Compare
Source](https://github.com/pmndrs/react-three-fiber/compare/v8.16.4...v8.16.5)

#### What's Changed

- Use new JSX runtime by [@&#8203;eps1lon](https://github.com/eps1lon)
in
[https://github.com/pmndrs/react-three-fiber/pull/3256](https://github.com/pmndrs/react-three-fiber/pull/3256)
- fix: Correct JSX module augmentation for React 19 types by
[@&#8203;eps1lon](https://github.com/eps1lon) in
[https://github.com/pmndrs/react-three-fiber/pull/3258](https://github.com/pmndrs/react-three-fiber/pull/3258)

#### New Contributors

- [@&#8203;eps1lon](https://github.com/eps1lon) made their first
contribution in
[https://github.com/pmndrs/react-three-fiber/pull/3256](https://github.com/pmndrs/react-three-fiber/pull/3256)

**Full Changelog**:
pmndrs/react-three-fiber@v8.16.4...v8.16.5

###
[`v8.16.4`](https://github.com/pmndrs/react-three-fiber/releases/tag/v8.16.4)

[Compare
Source](https://github.com/pmndrs/react-three-fiber/compare/v8.16.3...v8.16.4)

#### What's Changed

- fix: createPortal effect merging stale data from initial render by
[@&#8203;notrabs](https://github.com/notrabs) in
[https://github.com/pmndrs/react-three-fiber/pull/3255](https://github.com/pmndrs/react-three-fiber/pull/3255)

#### New Contributors

- [@&#8203;notrabs](https://github.com/notrabs) made their first
contribution in
[https://github.com/pmndrs/react-three-fiber/pull/3255](https://github.com/pmndrs/react-three-fiber/pull/3255)

**Full Changelog**:
pmndrs/react-three-fiber@v8.16.3...v8.16.4

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled because a matching PR was automerged
previously.

♻ **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/r4ai/r4ai.dev).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4zNTEuMiIsInVwZGF0ZWRJblZlciI6IjM3LjM1MS4yIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6W119-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
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.

2 participants