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

[feat] update for R3F v9 #1941

Open
wants to merge 11 commits into
base: v10
Choose a base branch
from
Open

[feat] update for R3F v9 #1941

wants to merge 11 commits into from

Conversation

krispya
Copy link
Member

@krispya krispya commented Apr 29, 2024

This is untested. It is a quick conversion of Drei v9 to use R3F v9 without any typing errors.

# Conflicts:
#	src/core/Environment.tsx
#	src/core/useEnvironment.tsx
Copy link

vercel bot commented Apr 29, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
drei ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 22, 2024 6:40pm

@krispya
Copy link
Member Author

krispya commented Apr 29, 2024

@joshuaellis I'd like to appeal to you about upgrading react-spring for R3F v9. :)

Copy link

codesandbox-ci bot commented May 8, 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.

@krispya
Copy link
Member Author

krispya commented May 8, 2024

Now builds. Known issues are with components that access R3F internals like RenderTexture.

@joshuaellis
Copy link
Member

@joshuaellis I'd like to appeal to you about upgrading react-spring for R3F v9. :)

Sorry I missed this. Do you know what the impact looks like? I can probably find some time for you haha

@krispya
Copy link
Member Author

krispya commented May 8, 2024

@joshuaellis I'd like to appeal to you about upgrading react-spring for R3F v9. :)

Sorry I missed this. Do you know what the impact looks like? I can probably find some time for you haha

I believe it is just types, maybe related to this: https://react.dev/blog/2024/04/25/react-19-upgrade-guide#the-jsx-namespace-in-typescript

We also moved types for R3F to ThreeElements.

@krispya krispya changed the base branch from master to v10 May 19, 2024 01:50
# Conflicts:
#	src/core/Text.tsx
package.json Outdated Show resolved Hide resolved
Co-authored-by: Cody Bennett <[email protected]>
@krispya
Copy link
Member Author

krispya commented May 21, 2024

@CodyJasonBennett I'm stuck again getting the test to run. Some kind of source map issue.

package.json Outdated Show resolved Hide resolved
Co-authored-by: Cody Bennett <[email protected]>
Copy link

Thank you for your work on this pull request! We’re marking this PR as stale to encourage a final push to bring it across the finish line.

Any update, comment, or change to the PR will automatically remove the stale label so it stays active in our workflow.

If you need any help, don’t hesitate to connect with us on Discord for feedback and support. This PR will close automatically soon if there’s no further activity, but we’re here to help if you’re interested in getting it merged. Thank you for your contribution!

@github-actions github-actions bot added the Stale Inactive issue label Oct 30, 2024
Copy link

github-actions bot commented Nov 7, 2024

We’re closing this pull request to help manage our workload and focus on active contributions. We really appreciate the effort you put into this, and we’d be happy to revisit it anytime you’re ready.

Please feel free to re-open this PR or reach out on Discord if you’d like to continue — our community is always here to support you. Thanks again for your contribution and for helping us stay organized!

@github-actions github-actions bot closed this Nov 7, 2024
@github-actions github-actions bot removed the Stale Inactive issue label Nov 8, 2024
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.

3 participants