Skip to content

upgrade React Navigation to v7 #414

Merged
PierreBresson merged 5 commits intoNMF-earth:mainfrom
hb2708:nmf-app-408
Dec 17, 2025
Merged

upgrade React Navigation to v7 #414
PierreBresson merged 5 commits intoNMF-earth:mainfrom
hb2708:nmf-app-408

Conversation

@hb2708
Copy link
Copy Markdown
Contributor

@hb2708 hb2708 commented Dec 17, 2025

✅ I have read the contributing file

Summary

Needed to complete #408, I will call this part 1 for it

Changelog

  • upgrade React Navigation to v7

Demo

Simulator.Screen.Recording.-.iPhone.17.Pro.-.2025-12-17.at.21.34.44.mov

- @react-navigation/native: 6.0.13 → 7.1.25
- @react-navigation/stack: 6.3.3 → 7.6.12
- @react-navigation/bottom-tabs: 6.4.0 → 7.8.12
- @react-navigation/material-top-tabs: 6.3.0 → 7.4.10
- Remove react-native-tab-view (now bundled in v7)
- Update Expo packages to SDK 54 compatible versions
React Navigation v7 requires all navigators to have unique id props.

Added ids:
- AppStack, RootStack, ModalStack, BottomTab
- ActStack, ActTab, AddEmissionStack, BudgetStack
- EmissionsStack, SettingsStack
- InfoModalStack, PeriodicityModalStack
Replace headerBackTitleVisible: false with headerBackButtonDisplayMode: 'minimal'
for React Navigation v7 compatibility.

Affected screens:
- About, Act, ActDetail, AddEmission, BarCodeScan
- CategorySelection, EmissionItem, Faq, Languages
- MonthlyBudget, MonthlyEmissions, MyData, MyLocation
- Notifications, RecurringEmissions, SubCategorySelection, SupportUs
@PierreBresson
Copy link
Copy Markdown
Member

Works well, except for this error when adding a new emission from Add tab :

The action 'NAVIGATE' with payload {"name":"Emissions","params":{}} was not handled by any navigator.

Do you have a screen named 'Emissions'?

If you're trying to navigate to a screen in a nested navigator, see https://reactnavigation.org/docs/nesting-navigators#navigating-to-a-screen-in-a-nested-navigator.

If you're using conditional rendering, navigation will happen automatically and you shouldn't navigate manually, see.

This is a development-only warning and won't be shown in production.
[React] 'The action \'NAVIGATE\' with payload {"name":"Emissions","params":{}}
was not handled by any navigator.\n\nDo you have a screen named
\'Emissions\'?\n\nIf you\'re trying to navigate to a screen in a nested
navigator, see
https://reactnavigation.org/docs/nesting-navigators#navigating-to-a-screen-in-a-nested-navigator.\n\nIf
you\'re using conditional rendering, navigation will happen automatically and
you shouldn\'t navigate manually, see.\n\nThis is a development-only warning and
won\'t be shown in production.', { [Stack] name: 'Stack' }

…a `EmissionsNavigator` with explicit parameters.
@hb2708
Copy link
Copy Markdown
Contributor Author

hb2708 commented Dec 17, 2025

@PierreBresson Should be gone now

@PierreBresson
Copy link
Copy Markdown
Member

PierreBresson commented Dec 17, 2025

@hb2708 indeed, merging thanks ✌️

@PierreBresson PierreBresson merged commit a1b7e52 into NMF-earth:main Dec 17, 2025
1 check passed
@hb2708 hb2708 deleted the nmf-app-408 branch December 18, 2025 04:50
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