Unfork NetworkIOAgent.cpp destructor and enableAgent API changes from 0.85 nightly integration
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 38/100
Piste de recherche
Commencez par vnext/ReactCommon/TEMP_UntilReactCommonUpdate/jsinspector-modern/NetworkIOAgent.cpp et comparez-le avec le commit React Native lié dans l’issue, en utilisant la PR d’intégration #16245 comme contexte. C’est terminé lorsque l’override inclut le destructeur et utilise enableAgent()/disableAgent() de manière cohérente avec l’API upstream.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Problem Description
During the integration of RN 0.85.0-nightly-20260128-36f07a1b2 (#16245), the upstream NetworkIOAgent API changed from enable()/disable() to enableAgent()/disableAgent(). A destructor was also added to properly clean up the network agent. These changes were applied to the existing override file.
Note: The existing override (tracked by #13587) covers type/namespace fixes. This issue tracks the new destructor and API adaptation.
Current Fix
Updated the existing patched copy at vnext/ReactCommon/TEMP_UntilReactCommonUpdate/jsinspector-modern/NetworkIOAgent.cpp with the destructor and new API calls.
Link to the integration PR where the error originated
#16245
Link to react-native commit
https://github.com/facebook/react-native/compare/f15985f4f...36f07a1b2
Forked files
No response
Upstream PR in react-native
No response
Build Configuration
None
- Langage dominant
- C++
- Étoiles
- 17.4k
- Forks
- 1.2k
- Merge moyen
- 1 j 20 h
- PR mergées (30 j)
- 37
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de microsoft/react-native-windows
-
Needs: Triage :mag:
Difficulté 2/5 1-2 jours Accessibilité débutants 64/100
microsoft/react-native-windows#16527 ·
Les mainteneurs répondent en général sous 1 jour
-
Fabric text is drawn with ClearType onto transparent composition surfaces, fringing thin glyphsOuverteNeeds: Triage :mag:
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
microsoft/react-native-windows#16340 ·
Les mainteneurs répondent en général sous 1 jour
-
bug Needs: Triage :mag:
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
microsoft/react-native-windows#16321 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Integration Follow-up Needs: Triage :mag:
Difficulté 4/5 3-5 jours Accessibilité débutants 42/100
microsoft/react-native-windows#16522 ·
Les mainteneurs répondent en général sous 1 jour
-
bug Needs: Triage :mag:
Difficulté 3/5 1-2 jours Accessibilité débutants 48/100
microsoft/react-native-windows#16479 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de microsoft/react-native-windows
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
pocoproject/poco#5626 ·
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
-
Issue-Bug Needs-Triage
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
microsoft/winget-cli#6645 ·
Les mainteneurs répondent en général sous 2 jours
-
cleanup help wanted
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
ESPresense/ESPresense#2602 ·
Les mainteneurs répondent en général sous 1 jour