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: remove mountWorkInProgressOffscreenFiber function #31473

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

Collection50
Copy link

@Collection50 Collection50 commented Nov 10, 2024

Summary

I deleted mountWorkInProgressOffscreenFiber.

The prop type of createFiberFromOffscreen is now specified as OffscreenProps, not any.

Therefore, I thought mountWorkInProgressOffscreenFiber was an unnecessary function.

image image

How did you test this change?

Copy link

vercel bot commented Nov 10, 2024

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

Name Status Preview Comments Updated (UTC)
react-compiler-playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 10, 2024 3:11pm

Copy link

This pull request has been automatically marked as stale. If this pull request is still relevant, please leave any comment (for example, "bump"), and we'll keep it open. We are sorry that we haven't been able to prioritize reviewing it yet. Your contribution is very much appreciated.

@github-actions github-actions bot added the Resolution: Stale Automatically closed due to inactivity label Feb 10, 2025
@Collection50
Copy link
Author

Thank you so much for your great contribution to the Javascript ecosystem.

I think this PR is still a valid PR. I'd appreciate it if you could check it :)

@Collection50
Copy link
Author

Hello, @sebmarkbage

Hi, I'd really appreciate it if you could check that PR.

Have a nice day !

@github-actions github-actions bot removed the Resolution: Stale Automatically closed due to inactivity label Feb 10, 2025
@Collection50
Copy link
Author

Hello, @eps1lon !

I think this PR is still a valid, it'd be nice if you could check it out !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants