Skip to content

RFC0929: Removal of the Legacy Architecture of React Native#929

Open
cortinico wants to merge 13 commits into
mainfrom
nc/legacy-arch-removal
Open

RFC0929: Removal of the Legacy Architecture of React Native#929
cortinico wants to merge 13 commits into
mainfrom
nc/legacy-arch-removal

Conversation

@cortinico

@cortinico cortinico commented Aug 29, 2025

Copy link
Copy Markdown
Member

Good night, sweet prince

This RFC contains the timeline and steps we're going to take for the removal of the Legacy Architecture. Any feedback is more than welcome, especially if you're on New Arch and you're still using one of those APIs listed for removal.

You can view a rendered version of the RFC here

@cortinico cortinico requested a review from cipolleschi August 29, 2025 13:47
@cortinico cortinico changed the title RFC0928: Removal of the Legacy Architecture of React Native RFC0929: Removal of the Legacy Architecture of React Native Aug 29, 2025

@cipolleschi cipolleschi left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

mostly nits, but there are a couple of points worth expanding.

Comment thread proposals/0929-legacy-architecture-removal.md Outdated
Comment thread proposals/0929-legacy-architecture-removal.md Outdated
Comment thread proposals/0929-legacy-architecture-removal.md Outdated
Comment thread proposals/0929-legacy-architecture-removal.md
Comment thread proposals/0929-legacy-architecture-removal.md Outdated
Comment thread proposals/0929-legacy-architecture-removal.md Outdated
Comment thread proposals/0929-legacy-architecture-removal.md Outdated
cortinico and others added 5 commits September 1, 2025 09:09
Co-authored-by: Riccardo Cipolleschi <riccardo.cipolleschi@gmail.com>
Co-authored-by: Riccardo Cipolleschi <riccardo.cipolleschi@gmail.com>
Co-authored-by: Riccardo Cipolleschi <riccardo.cipolleschi@gmail.com>
Co-authored-by: Riccardo Cipolleschi <riccardo.cipolleschi@gmail.com>
Co-authored-by: Riccardo Cipolleschi <riccardo.cipolleschi@gmail.com>

@mateoguzmana mateoguzmana left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Amazing – just a few nits I noticed while reading it through.

Comment thread proposals/0929-legacy-architecture-removal.md Outdated
Comment thread proposals/0929-legacy-architecture-removal.md Outdated
Comment thread proposals/0929-legacy-architecture-removal.md Outdated
cortinico and others added 3 commits September 1, 2025 12:06
Co-authored-by: Mateo Guzmán <info@mateoguzman.net>
Co-authored-by: Mateo Guzmán <info@mateoguzman.net>
Co-authored-by: Mateo Guzmán <info@mateoguzman.net>
Comment thread proposals/0929-legacy-architecture-removal.md Outdated
Comment thread proposals/0929-legacy-architecture-removal.md Outdated
Comment thread proposals/0929-legacy-architecture-removal.md Outdated
- [ ] `RCTWrapperViewControllerHostingView`
- [ ] `RCTWrapperViewManager`
- [ ] `systrace_arg_t`
- [ ] `UIScrollViewDelegate`

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I saw that react-native-webview is using this, but I can't find in the deprecation notices if there is a replacement. Will we document the new alternatives?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I think UIScrollViewDelegate is a mistake. That class comes from UIKit, so we can't deprecate it ourselves!

@Sarvesh0007 Sarvesh0007 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

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.

9 participants